Repository navigation
decisions: PLATFORM-PIN moves to platform-sw 131e500 (2.0 base) - #74
BotshareAI wants to merge 1 commit into
Conversation
P-03 rev20.1 item 25 D7: the combined robot model in openamr-upperbody-sw is now tested against platform-sw 131e500fc4e5afc9f648cbd7dbf5b416280761fb. Validated by openamr-upperbody-sw PR #21 (A6 793 executed / 0 failed, A7 246 executed / 0 failed at a0f7dd6). The dfd44a5 pin of 20 Oct 2026 moves to supersedes; P-03-rev20.1 is added to sources. bash rollout/verify.sh: 278 of 278 executed, PASS. check_decisions.py --validate-only: 39 loaded, valid. Assisted-by: Claude Code Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DZnPENZHgKq9tLAj9DLAnR Signed-off-by: Alex Reznichenko <info@botshare.ai>
panthera-momagdii
left a comment
There was a problem hiding this comment.
Focused integration/provenance review at head 30b46b2d90ba1c7646c841f100e7eb6080b27785, changing only decisions.yaml. I inspected the exact register diff, shared AGENTS/decision rules, the linked upperbody #21 validation and the current parallel governance #73 diff. The register checker and Repository quality results for #74 are green, but this is not an approval.
[P2] Coordinate with #73 before making the new PLATFORM-PIN authoritative. Both PRs branch from main at 71c439e and both add sources.P-03-rev20.1 at the same insertion point, with different source-title/provenance text. #73 also changes in_force.source from rev19.6 to rev20.1, and currently has an outstanding changes-requested review concerning overly broad contradiction-checker exceptions. #74 does not mention #73 in its Integration Gate and, merged by itself, would cite rev20.1 for the pin while in_force still says rev19.6. Please put #73 first in the merge order; after its checker and owner review are resolved, bring that exact reviewed head into this branch with a normal merge commit (no force-push), keep one consistent P-03-rev20.1 source definition, retain the in-force date/source and this PR's PLATFORM-PIN history, and rerun bash rollout/verify.sh and check_decisions.py --validate-only on the combined tree. If the platform lead prefers a single register PR, consolidate there instead and close the redundant one only by owner action.
Primary decision still requires a human source check: the P-03 rev20.1 item 25 D7 document is not in the repository. #21's tested platform SHA 131e500 is independently evidenced, but successful static/mock and simulation runs are not approval of the source wording or hardware. Keep the register PR draft until Alex/software lead confirms the exact decision and consumer/owner handoff. No repository code, register values or settings were changed by this review. Not run locally: the full register verifier, A6/A7, hardware. AI-assisted focused review.
Summary
Updates the PLATFORM-PIN entry in
decisions.yaml(P-03 rev20.1 item 25, decision D7): the combined robot model in openamr-upperbody-sw is tested against platform-sw131e500fc4e5afc9f648cbd7dbf5b416280761fb(the 2.0 base), dated 2026-10-30. The dfd44a5 pin of 20 Oct 2026 moves tosupersedes. Source: "P-03 rev20.1 item 25 D7; validated by openamr-upperbody-sw PR #21 (A6 793 executed / 0 failed, A7 246 executed / 0 failed at a0f7dd6)".P-03-rev20.1is added undersources(the validator requires every cited document to be listed). Check patterns unchanged. Scope:decisions.yamlonly.Work package
Contract/decision update for P-03 rev20.1 item 25 D7; consumer close-out is openAMRobot/openamr-upperbody-sw#10 (A6 + A7, roadmap OA-12).
Integration Gate
config/upstream_pins.yamlopenamr_platform_sw) already carries 131e500 on the Update demo section to include release version #21 branch; no other consumer states the pin value as text covered by a check pattern.Tests
No checker or pattern change, so no new test. The register is exercised by the checker unit tests in
rollout/verify.sh: before addingP-03-rev20.1tosources, the same run failed (validator: "PLATFORM-PIN: source document 'P-03-rev20.1' not listed under sources"); with it, 278 of 278 executed, 0 skipped, PASS. main 71c439e: 278 of 278, PASS.Evidence
Base SHA: 71c439e
Head SHA: 30b46b2
Dependencies
None.
Safety impact
None.
STATE.md
No change: this repository has no STATE.md.
Not verified
AI disclosure
Claude Code, at Alex Reznichenko's request, produced the register edit, commit and this description. Draft until the owner writes adopt, adapt or reject.
Contribution terms
Generated by Claude Code