Skip to content

fix: resolve deprecation warning for apiKey value#18

Merged
patlux merged 2 commits into
patlux:mainfrom
plumj-am:patch/PlumJam-vlqsqmzoryrp
Jun 16, 2026
Merged

fix: resolve deprecation warning for apiKey value#18
patlux merged 2 commits into
patlux:mainfrom
plumj-am:patch/PlumJam-vlqsqmzoryrp

Conversation

@plumj-am

@plumj-am plumj-am commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Shown on startup:

Deprecation warning: registerProvider("commandcode") apiKey value "COMMANDCODE_API_KEY" is treated as a legacy environment variable reference. This will no longer be detected as an environment variable reference in a future release. Pass "$COMMANDCODE_API_KEY" instead.

Support for previous setup kept to avoid a breaking change.

Signed-off-by: PlumJam <git@plumj.am>
@plumj-am

Copy link
Copy Markdown
Contributor Author

@patlux bump :)

@patlux

patlux commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Validated this locally with pi 0.78.0.

  • On main, COMMANDCODE_API_KEY=mock-key pi --no-extensions -e ./index.ts --list-models commandcode reproduces the deprecation warning.
  • On this PR branch, the same command lists models without the warning.
  • npm run typecheck passed.
  • npm run test:stream passed.

The previous npm audit failure was a pre-existing baseline issue on main, not caused by this PR. It has since been fixed separately in #21, so I am updating this branch with latest main to re-run CI.

@patlux patlux mentioned this pull request Jun 16, 2026
4 tasks
@patlux

patlux commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Thanks @plumj-am! I validated the fix locally with pi 0.78.0, updated the branch with latest main, and CI is green now. Appreciate the compatibility fix and the nudge 🙌

@patlux patlux merged commit 88d8706 into patlux:main Jun 16, 2026
10 checks passed
@plumj-am plumj-am deleted the patch/PlumJam-vlqsqmzoryrp branch June 16, 2026 12:49
@patlux

patlux commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Shipped in pi-commandcode-provider@0.4.1 / GitHub release v0.4.1.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants