chore(deps): migrate @mariozechner -> @earendil-works + tighten parameter types#17
Open
av1155 wants to merge 1 commit into
Open
chore(deps): migrate @mariozechner -> @earendil-works + tighten parameter types#17av1155 wants to merge 1 commit into
av1155 wants to merge 1 commit into
Conversation
av1155
force-pushed
the
chore/audit-deps-refresh
branch
from
649e858 to
36f218d
Compare
… casts - Migrate peerDependencies and src/index.ts imports from @mariozechner/pi-coding-agent (deprecated upstream) to @earendil-works/pi-coding-agent ^0.74.0. - Add @earendil-works/pi-coding-agent ^0.74.0 to devDependencies so the type imports resolve standalone. - Bump @types/node ^22.15.21 -> ^25.7.0. - Drop `as any` from the five Type.Object schemas passed into pi.registerTool. The cast was load-bearing under the old @mariozechner typings, but @earendil-works's ToolDefinition<TParams extends TSchema> infers Static<TParams> correctly when the schema is left typed, which lets the execute(params, ...) body keep the params shape instead of falling through to unknown. After this change npm install reports 0 deprecations from this repo's direct or peer dependencies (node-domexception remains transitive), tsc --noEmit is clean, and the 73 existing node:test cases all pass.
av1155
force-pushed
the
chore/audit-deps-refresh
branch
from
36f218d to
2a33c37
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
npm installat v1.3.2 surfaces four@mariozechnerdeprecation warnings (transitive via the@mariozechner/pi-coding-agentpeer dep). The@mariozechnernamespace is deprecated upstream;@earendil-works/pi-*at v0.74.0 is the canonical replacement.Changes
peerDependencies:@mariozechner/pi-coding-agentswapped for@earendil-works/pi-coding-agent.devDependencies: add@earendil-works/pi-coding-agentat^0.74.0so type imports resolve standalone.src/index.ts: imports updated to the new namespace.@types/node:^22.15.21to^25.7.0.as anyfrom the fiveType.Objectschemas passed intopi.registerTool. The cast was load-bearing under the old@mariozechnertypings;@earendil-works'sToolDefinition<TParams extends TSchema>infersStatic<TParams>correctly when the schema is left typed, which keeps theexecute(params, ...)body strongly typed instead of falling through tounknown.Verification
npm install: 0 deprecations from this repo's direct or peer dependencies. Onlynode-domexceptionremains, transitive throughfetch-blob.tsc --noEmit: clean.npm test(node --test --import tsx src/**/*.test.ts): 73 tests pass.Held back
typescript:^5.9.3to^6.0.3. The TS 6 stricter lib types unmask a fewunknownpropagations elsewhere in the file that would need separate cleanup; held back from this PR so the dep refresh stays scoped.