Skip to content

Commit fb9c339

Browse files
aorumbayevclaude
andcommitted
fix(cd): build @kagan/shared-api-client before web-build step
dist/ is gitignored so pnpm install alone doesn't produce it. Add an explicit build step in cd.yaml to mirror the fix already applied to the ci.yml test-web job. Also annotate the find callback parameter in use-session-stream.ts to guard against implicit-any if the shared package ever fails to resolve again. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7ab5666 commit fb9c339

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

.github/workflows/cd.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,9 @@ jobs:
8181
- name: Install JavaScript workspace dependencies
8282
run: pnpm install --frozen-lockfile
8383

84+
- name: Build shared API client
85+
run: pnpm --filter @kagan/shared-api-client run build
86+
8487
- name: Build bundled web UI
8588
run: uv run poe web-build
8689

packages/web/src/lib/hooks/use-session-stream.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ function summarizeEvent(event: WireEvent): { text: string; status: StreamedStatu
6464
}
6565

6666
if (renderable) {
67-
const bodyLine = renderable.body.split('\n').find((line) => line.trim().length > 0);
67+
const bodyLine = renderable.body.split('\n').find((line: string) => line.trim().length > 0);
6868
text = bodyLine?.trim() ?? renderable.title;
6969
} else {
7070
text = event.type;

0 commit comments

Comments
 (0)