Skip to content

Point agents at the shared Python and Odoo style (cbusillo/codex-skills#2381) - #58

Merged
2 commits merged into
mainfrom
work/style-pointer-2381
Oct 21, 2026
Merged

2 commits merged into
mainfrom
work/style-pointer-2381

Conversation

@shiny-code-app

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

Copy link
Copy Markdown

The shared style reference merged as codex-skills#2383.

This adds a pointer to AGENTS.md that points them at the shared Python and Odoo style page, as codex-skills#2381 Bring Chris's code style forward asks.

Keeps the library Python range and public API compatibility authoritative when shared Odoo runtime guidance does not apply.

Verification: this changes agent instructions only, with no code, so there are no tests or inspection to run.

Client test notes

Nothing for the Client to test: agent instruction pointer only

@shiny-code-app

Copy link
Copy Markdown
Author

POINTERS-C1 stopped at claim preflight. Exact refusal (claim_conflict, write_outcome: not_started):

Another worker or ambiguous ownership evidence holds this issue; preserve it for owner review

The helper identifies the existing work/style-pointer-2381 branch and /Volumes/Developer-Artifacts/worktrees/printnode_community/style-pointer-2381 worktree as current_or_stale ownership evidence, with no verified native owner. No claim, worktree, edit, push, review attestation or ready handoff was made.

Read-only checks: the PR's pointer targets skills/references/python-odoo-style.md, now present on codex-skills main after #2383 merged. Current main here has no pointer, so this PR is not redundant. At head 4c5d42575efd8ab999e7c53498486747016a24ee: 9 check runs; 0 failing, 0 pending; 0 unfinished workflows.

Next action: Supervisor coordinates the original source session's supported ownership handoff or recovery; the next agent then retries this PR-only claim before editing the PR notes, running and weighing the review, and posting checked ready. The hold text in the body is obsolete; repositories feeding a Client release need ## Client test notes with Nothing for the Client to test: agent instruction pointer only.

@shiny-code-app

Copy link
Copy Markdown
Author

Claimed by codex-pointers-c1

Session: 01a15c4b-370f-7092-9a27-0b62a105523d
Branch: work/pointers-c1-58
Claimed at: 2026-10-20T22:52:32-04:00
Next action: Finish pointer PR: correct notes, weigh independent review, verify checks and hand off checked ready

Admission: spare-capacity; cbusillo/codex-skills#1043

Wait resolution: Full PR discussion reviewed. codex-skills#2383 merged 2026-10-20T21:36:51Z. Supervisor confirmed STYLE-C1 ended and retired its clean original worktrees/local branches; remote PR branches preserved. No other person/event hold.

Work record

@shiny-code-app

Copy link
Copy Markdown
Author

Review result: weighed
Head: 84b8431

Anthropic claude-opus-5-5[1m] reviewed the final committed instructions and actual diff, following the ten-PR initial review. Acted on the Python-floor finding: package-supported Python and public API compatibility take precedence over the Odoo image target and newer typing features. Final review: no defects. Declined the remaining notes as pointer defects: missing Ruff setup is existing tooling coverage, with no new enforcement gate claimed here; nothing in this PR directs removing existing public docstrings or renaming public APIs. No extra policy text is needed.

No tests added and no literals, wording or config text pinned. Local validation: git diff --check passed; MCP canonical Python sample parsed successfully. Runtime tests and IDE inspection not run: Markdown instructions and example only, no runtime code paths affected.

@shiny-code-app

Copy link
Copy Markdown
Author

Ready for the merge train

Head: 84b8431

@shiny-code-app

Copy link
Copy Markdown
Author

Released claim 6306827981

Handoff from codex-pointers-c1
Source claim: 6306827981
Session: 01a15c4b-370f-7092-9a27-0b62a105523d
Retained PR: #58
Retained remote branch: work/style-pointer-2381
Head: 84b8431
Checked ready: #58 (comment)

Retained task worktree: /Volumes/Developer-Artifacts/worktrees/printnode_community/pointers-c1-58
Retained task branch: work/pointers-c1-58
The worktree is clean and its committed head is pushed to the PR. Retain until landing; the landing agent uses dev-worktree retire as its lock requires.

Current Status

Next action: Supervisor's assigned merge-train agent lands this reviewed, green current-head PR and performs After A Merge. No Client acceptance is requested for this agent-instruction pointer. No further implementation is needed from this session; a later head requires fresh checks, weighed review metadata and ready.
Last verified: POINTERS-C1, October 20, 2026. All checks settled; current-head reviewed ready handoff is linked above. Original ownership conflict is resolved.

@shiny-code-app

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

Copy link
Copy Markdown
Author

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

Launchplane landed this pull request through protected batch PR #59.

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 21, 2026
@shiny-code-app shiny-code-app Bot mentioned this pull request Oct 21, 2026
@shiny-code-app shiny-code-app Bot closed this pull request by merging all changes into main in 6d06965 Oct 21, 2026
@shiny-code-app
shiny-code-app Bot deleted the work/style-pointer-2381 branch October 21, 2026 03:08
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