Skip to content

Declare focused local tests and the complete CI suite - #268

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

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

Conversation

@shiny-code-app

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

Copy link
Copy Markdown
Contributor

Devkit CI already runs the complete unittest suite. Declare a selected test-file command for local work and retain the existing full discovery command as the CI gate, adopting Q700.

README replaces the unqualified full-suite command with local selection and affected safety-module guidance. Existing live Odoo and tenant consumer-build requirements remain in place.

Validation: JSON parsing, Ruff format/lint and diff checks passed. Anthropic review was weighed; sibling safety modules were included and the duplicate CI command replaced with its metadata reference. No runtime code or tests changed. IDE inspection was not run for this JSON/Markdown change under the brief's prohibition. Full tests and tenant build-only checks must pass before ready.

Refs cbusillo/codex-skills#1884

Client test notes

Nothing for the Client to test: this changes engineering test selection metadata and guidance only. Runtime behavior and Client workflows are unchanged.

@shiny-code-app

Copy link
Copy Markdown
Contributor Author

Review result: weighed
Head: 9833768

Anthropic claude-opus-5-5[1m] completed the review (model confirmed by CLI; effort unknown). I acted on both low-severity findings: added the sibling override-guard, serialization and addon-credential test modules to the selection guidance, and replaced the repeated CI command with its metadata reference. JSON parsing, diff checks and existing Ruff format/lint checks passed. The review verified that the required test job runs the complete discovery command and no consumer reads the removed default key. No tests were added and no findings remain unresolved. These local documentation corrections require no further model run under the review policy. Existing tenant consumer-build checks and current-head CI still gate ready.

@shiny-code-app

Copy link
Copy Markdown
Contributor Author

Both required tenant consumer-build checks passed on attempt 1, and their retained consumer-build-check.json receipts were verified before handoff.

Tenant Run Tenant source Runtime/devkit commit Platform
CM website build-only check a9ad996228e6661a8a04e8644d163eed7b5652a9 9833768c8c6cf862e3d696f330d0f294527d5ed5 linux/amd64
OPW build-only check 6af6b4806c73def3ac161123dc460a466b888fc9 9833768c8c6cf862e3d696f330d0f294527d5ed5 linux/amd64

The receipts include dependency provenance and lock evidence. These were existing artifact-build gates: no image publication, deploy, data change, grant or Client contact. Nothing needed from Chris. Checked ready follows for this reviewed head.

@shiny-code-app

Copy link
Copy Markdown
Contributor Author

Ready for the merge train

Head: 9833768

@shiny-code-app

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

Copy link
Copy Markdown
Contributor 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 af3f468 into main Oct 23, 2026
5 checks passed
@shiny-code-app
shiny-code-app Bot deleted the work/q700-1023-rollout branch October 23, 2026 12:58
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