Skip to content

Commit f7e2074

Browse files
committed
prerequisites and build fixes
1 parent 68c2ddc commit f7e2074

5 files changed

Lines changed: 15 additions & 8 deletions

File tree

frontend/src/components/entity-forms/prerequisite-form.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -119,10 +119,7 @@ export const PrerequisiteForm = Shade<PrerequisiteFormProps>({
119119
options={typeOptions}
120120
value={props.initial?.type ?? ''}
121121
placeholder="Select a prerequisite type..."
122-
onchange={(ev) => {
123-
const value = (ev.target as HTMLSelectElement).value as PrerequisiteType
124-
setSelectedType(value)
125-
}}
122+
onValueChange={(value) => setSelectedType(value as PrerequisiteType)}
126123
/>
127124

128125
{(selectedType === 'node' || selectedType === 'yarn') && (

frontend/src/components/prerequisite-table.tsx

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -201,6 +201,16 @@ export const PrerequisiteTable = Shade<PrerequisiteTableProps>({
201201

202202
return (
203203
<div>
204+
<div style={{ display: 'flex', justifyContent: 'flex-end', marginBottom: '8px' }}>
205+
<Button
206+
variant="outlined"
207+
size="small"
208+
onclick={() => setIsCreating(true)}
209+
startIcon={<Icon icon={icons.plus} size="small" />}
210+
>
211+
Add Prerequisite
212+
</Button>
213+
</div>
204214
<DataGrid<Prerequisite, PrerequisiteColumn>
205215
columns={['name', 'type', 'status', 'actions']}
206216
findOptions={findOptions}

service/src/app-models/prerequisites/actions/check-prerequisite-action.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
99
import { runCheck, CheckPrerequisiteAction } from './check-prerequisite-action.js'
1010

1111
const execFileMock = vi.hoisted(() =>
12-
vi.fn<[string, string[], { timeout: number }], Promise<{ stdout: string; stderr: string }>>(),
12+
vi.fn<(cmd: string, args: string[], options: { timeout: number }) => Promise<{ stdout: string; stderr: string }>>(),
1313
)
1414

1515
vi.mock('child_process', () => ({

service/src/app-models/stacks/actions/import-export-actions.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import { ImportStackAction } from './import-stack-action.js'
2020
class AutoIncrementStore<T extends { id: number }> extends InMemoryStore<T, 'id'> {
2121
private nextId = 1
2222

23-
async add(...items: T[]): Promise<T[]> {
23+
public async add(...items: T[]) {
2424
const withIds = items.map((item) => ({
2525
...item,
2626
id: item.id || this.nextId++,

service/src/mcp/mcp-server.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -280,11 +280,11 @@ export const createMcpRequestHandler = (injector: Injector, sessionManager: McpS
280280
const mcp = createMcpServer(injector, elevated)
281281
await mcp.connect(transport)
282282

283+
await transport.handleRequest(req, res)
284+
283285
if (transport.sessionId) {
284286
sessionManager.register(transport.sessionId, transport)
285287
}
286-
287-
await transport.handleRequest(req, res)
288288
return
289289
}
290290

0 commit comments

Comments
 (0)