Skip to content

chore(deps): autoupdate (needs claude fix)#28

Open
github-actions[bot] wants to merge 3 commits into
mainfrom
chore/autoupdate-30135324351
Open

chore(deps): autoupdate (needs claude fix)#28
github-actions[bot] wants to merge 3 commits into
mainfrom
chore/autoupdate-30135324351

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

The dependency autoupdater failed during builds-and-checks.

Run log: https://github.com/dudko-dev/agent/actions/runs/30135324351

A Claude session has been dispatched to push fixes onto this branch
so the following commands all exit 0:

  npm run typecheck
  npm run format:check
  npm run build
  npm test

Claude will leave a status comment on this PR when it finishes.
PR-checks will re-run on each new commit.

@github-actions
github-actions Bot requested a review from siarheidudko July 25, 2026 00:02
@github-actions github-actions Bot added autoupdate Automated dependency update PRs needs-claude Needs Claude GitHub App to fix labels Jul 25, 2026
TypeScript 7.0.2 is a full compiler rewrite (native, Go-based) that
removes the classic ts.sys/createProgram API tsup's bundled
rollup-plugin-dts relies on for `dts: true` bundling, so `npm run
build` crashed with "Cannot read properties of undefined (reading
'useCaseSensitiveFileNames')". tsup 8.5.1 (latest) doesn't support
TS7 yet, so hold typescript at ^6.0.3 (dev-only, no product impact)
until it does.

Also applies a prettier 3.9.6 reformat of src/types.ts (new rule
default), picked up by format:check.
@claude

claude Bot commented Jul 25, 2026

Copy link
Copy Markdown

Autoupdate compatibility check — results

Initial run: 3 of 4 commands failed.

  • npm run typecheck — green on first run.
  • npm run format:checkfailed: prettier 3.9.6 (bumped from 3.8.4) reformats a union-type line in src/types.ts under a new rule default. Fixed by applying prettier --write (pure formatting, no logic change).
  • npm run buildfailed: typescript was bumped ^6.0.3^7.0.2. TypeScript 7 is a full compiler rewrite (native/Go-based); its npm package no longer exposes the classic ts.sys / createProgram API that tsup's bundled rollup-plugin-dts needs for dts: true bundling. Build crashed with Cannot read properties of undefined (reading 'useCaseSensitiveFileNames'). tsup 8.5.1 (already the latest release) doesn't support TS7 yet. Fixed by holding typescript back at ^6.0.3 in devDependencies — it's dev/build tooling only, no product/runtime impact, and typecheck/tests are unaffected by this pin.
  • npm test — showed 1 failing test (buildModelFromStage ... provider=anthropic, AI_InvalidArgumentError: baseURL must be a non-empty string) on the first run, but this was a false alarm caused by an ANTHROPIC_BASE_URL="" environment variable leaking into my local execution sandbox (unrelated to this repo's test.yml, which sets no such env var). Verified by re-running with that stray var removed: all 134 tests pass, including the anthropic one. No code change was needed for this; flagging in case it looks confusing in a local dev run — it should not reproduce in GitHub Actions CI.

Changes pushed: commit b0a26ac — reverted typescript devDependency to ^6.0.3 and reformatted src/types.ts per the new prettier version. No product logic changed.

Final local verification (clean env, no stray ANTHROPIC_BASE_URL): all four commands green — typecheck ✅, format:check ✅, build ✅, test (134/134) ✅.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autoupdate Automated dependency update PRs needs-claude Needs Claude GitHub App to fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant