Skip to content

ci: bump node-platform reusable-workflow pin (red conformance run was registry lag) - #3

Merged
HomenShum merged 2 commits into
mainfrom
claude/ci-conformance-fix
Jul 29, 2026
Merged

ci: bump node-platform reusable-workflow pin (red conformance run was registry lag)#3
HomenShum merged 2 commits into
mainfrom
claude/ci-conformance-fix

Conversation

@HomenShum

Copy link
Copy Markdown
Owner

Root cause of the red 2026-07-20 conformance run

The failing "Node Platform conformance" run on PR #2 was registry lag, not a NodeAgent defect. The run predated node-platform registering nodeagent.event-protocol and nodeagent.provider-pi in ownership.yaml, so the checker reported them as unknown concepts and flagged the two contract declarations as stale. Those concepts are on node-platform main now.

Proof: re-running the exact same failed run against today's registry passes — https://github.com/HomenShum/NodeAgent/actions/runs/29712393345 (now green, 4/4 contract declarations classified). Also reproduced locally with node-platform@ab7c9e69 as the registry root.

What this PR changes

Bumps the pinned reusable-workflow SHA from ef4ac8a (20 days stale) to ab7c9e69 (current node-platform main), picking up the CI gate fixes from node-platform #22/#23. Workflow interface (platform_ref, node_version) is unchanged; the registry checkout still floats at main by design.

Test count check

npm test (vitest) on main: 41 passing. On the PR #2 branch: 47 passing (that PR adds 6 tests). The "41 passing tests" claim is accurate for main.

🤖 Generated with Claude Code

HomenShum and others added 2 commits July 29, 2026 02:30
The 2026-07-20 red conformance run on PR #2 was registry lag, not a
NodeAgent defect: the run predated node-platform registering the
nodeagent.event-protocol and nodeagent.provider-pi concepts in
ownership.yaml. Re-running the same run against today's registry
passes 4/4 contract declarations (verified locally and on Actions).

This bump moves the pinned reusable-workflow SHA from ef4ac8a (20 days
stale) to ab7c9e69, picking up node-platform's CI gate fixes (#22/#23).
The workflow interface (platform_ref, node_version) is unchanged.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
node-platform main now registers nodeagent.event-protocol and
nodeagent.provider-pi with NodeAgent as owner, so the conformance
check requires them in canonicalFor. Verified locally against
node-platform@ab7c9e69: PASS, 2/2 declarations classified.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@HomenShum
HomenShum merged commit a0f1800 into main Jul 29, 2026
3 checks passed
@HomenShum
HomenShum deleted the claude/ci-conformance-fix branch July 29, 2026 09:40
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.

1 participant