Skip to content

Adopt focused local tests and the complete CI test command - #62

Merged
shiny-code-app[bot] merged 3 commits into
mainfrom
work/q700-1023-rollout
Oct 23, 2026
Merged

shiny-code-app[bot] merged 3 commits into
mainfrom
work/q700-1023-rollout

Conversation

@shiny-code-app

Copy link
Copy Markdown

Adopt Q700 with selected local pytest paths and the existing complete suite in CI. CI and Publish read the canonical test.ci command; remove the duplicate full local pytest run from release.verify while keeping its package checks.

Reconcile AGENTS, contributor, test and release guides. Local safety coverage follows the affected auth flow or mutated endpoint and its callers. README links remain usable in the PyPI description.

Validation: all 35 auth tests, build and twine package checks, JSON, actionlint and whitespace checks passed. Anthropic review was weighed; the endpoint-coverage guidance and package README links were corrected. No tests or publishing permissions change. Full suites run in CI before ready. JetBrains inspection NOT RUN under the brief's explicit IDE prohibition.

Client test notes

Nothing for the Client to test: engineering test metadata, CI command routing and developer guidance only. No publish, device or production action.

Refs cbusillo/codex-skills#1884

@shiny-code-app

Copy link
Copy Markdown
Author

Review result: weighed
Head: c79cef7

Anthropic claude-opus-5-5[1m] completed the read-only review (model confirmed by CLI; effort unknown). No blocking/high finding. Fixed the incomplete safety mapping by selecting complete modules for the affected auth flow or mutated endpoint and its callers. Fixed the catalog HEAD link and both PyPI README links. The workflow-metadata anchor was checked against the catalog source. No unresolved finding remains.

CI and Publish execute the same complete pytest suite through the canonical metadata command. Publish's tag/environment/trusted-publisher behavior is unchanged. Local focused auth coverage (35 tests), build/twine, JSON, actionlint and whitespace checks pass. No tests are added, no literal/config text is asserted, and no gate is weakened. These local documentation corrections implement the completed review; no repeat inference is needed. Complete current-head CI is required before ready. Inspection NOT RUN under the explicit brief prohibition.

@shiny-code-app

Copy link
Copy Markdown
Author

Ready for the merge train

Head: c79cef7

@shiny-code-app

shiny-code-app Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Author

Launchplane completed the merge-train step for this pull request.

Launchplane finished the merge-train step for this pull request.

Launchplane manages this comment and will update it as the train moves.

shiny-code-app Bot added a commit that referenced this pull request Oct 23, 2026
@shiny-code-app
shiny-code-app Bot merged commit 639e335 into main Oct 23, 2026
9 checks passed
@shiny-code-app
shiny-code-app Bot deleted the work/q700-1023-rollout branch October 23, 2026 15:21
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.

0 participants