Repository navigation
feat(core)!: require documented Provider API access (carry #164) - #203
Merged
Merged
Conversation
Carry Patrick Wozniak's PR #164, adapted to current main. Remove the internal generate inference fallback and its distributed client. Keep entitled Provider API inference, read-only quota/account reporting, credentials, persisted endpoint choices and native host pricing intact. Cover denial diagnostics, latest-request ownership, error redaction, terminal events and zero generate retries through real-host mocks. BREAKING CHANGE: Go-plan inference through the internal generate endpoint is removed. Use an account with documented Provider API entitlement. Read-only account/quota reporting remains available. Co-authored-by: Patrick Wozniak <email@patwoz.de>
Contributor
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Contributor
Memory benchmarkCompared base
Interpretation: No metric shows a clear Base-to-PR difference beyond its measured run-to-run variation. Extension overhead above pi baseline
Values are medians of 6 alternating, paired runs. The value after Environment: pi
|
This was referenced Nov 5, 2026
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
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.
Attribution and scope
Carries #164 by Patrick Wozniak (@patlux), adapted to current main after #202. The commit preserves Patrick's co-author credit. The original PR remains separate.
Breaking inference policy
/alpha/generateretry after403 upgrade_required, including the retired client, converters, schema/transcript modules and generate-only fixtures/tests.Validation
Fresh local validation on this exact content:
npm testwith Pi and OMP required: 430 Node tests, zero failures/cancellations/skips; Pi 1.5.0, OMP 18.16.0 and Bun 1.4.4.command-code@1.88.3; public pricing check passes without edits (95 advertised models checked).Review and limits
An independent read-only
deepseek/deepseek-flashreview with thinkingmaxreturned OK with notes, with no P0/P1 finding. The small cleanup/coverage notes were addressed and revalidated by the author; the final cleanup delta was not independently re-reviewed.Tests use synthetic credentials and mock upstreams, not paid/live plan probes. The editor LSP uses inferred settings for the entrypoint; the real-host compile gate above is authoritative. Existing unpinned actions/checkout-credential warnings in the catalog workflow are unchanged; this PR only redirects its source-path filter from the removed client to the router. This is not a globally-clean static-analysis claim.
Release
This carry intentionally leaves the package version at
0.7.12. The entitlement change targets a separate 0.8.0 release PR and the existing tag/OIDC release workflow; merging this PR does not publish to npm or update installed extensions.