Repository navigation
fix: canonicalize optional AI repository identity - #90
Merged
Merged
Conversation
Co-Authored-By: Codex <noreply@openai.com>
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.
The QSL lifecycle's printed strict report identifies four unmanaged QSP entries that still name AIAuditBridge. Use the canonical QuantStrategyLab/QuantAIService name in qsl.toml, pyproject.toml and the two uv.lock source literals.
Keep ai-gateway-client, pin 65c8bcf432f578f657c12d5dab0e16ebec0bef5a and optional ai unchanged. The lock was edited only for these repository literals, with no resolution or dependency upgrade.
Depends on QuantStrategyLab/QuantRuntimeSettings#719 for its bounded canonical identity checkout. After independent review, adopt QRS first and this metadata second. Neither single-repo intermediate bypasses the existing strict gate.
Validation: all three files parse as TOML and match the originals after only the approved literal replacement. Four synthetic cross-repo stages using these actual metadata bytes reproduce the original four strict errors and prove the combined candidate clean. Companion QRS tests: 44 methods pass, with 30 separate subcases; actionlint and bash syntax pass. No local dependency install, real AI/provider call or deployment. Draft pending independent review; no merge or workflow dispatch/rerun.