ci: sync the review workflow pair to the intent-routing activation release #14

Merged
jercik merged 1 commit from chore/router-13-review-workflows into main 2026-09-01 09:06:55 +00:00
Owner

Pre-staged for the intent-routing activation (cluster#354). The synced pair replaces the Router-major-to-axrun mapping with the literal @j4k/axrun@18.0.0 the release manifest pins.

Do not merge outside the activation window: this workflow fails against a live Router 12, and the fenced maintenance play byte-audits every admitted main against exactly this pair before mutating anything.

Pre-staged for the intent-routing activation (cluster#354). The synced pair replaces the Router-major-to-axrun mapping with the literal `@j4k/axrun@18.0.0` the release manifest pins. Do not merge outside the activation window: this workflow fails against a live Router 12, and the fenced maintenance play byte-audits every admitted main against exactly this pair before mutating anything.
ci: sync the review workflow pair to the intent-routing activation release
All checks were successful
PR Review / Prepare immutable review tools (pull_request_target) Has been skipped
PR Review / forgejo-review-approach-smart-1 generator (pull_request_target) Has been skipped
PR Review / forgejo-review-approach-luna-1 generator (pull_request_target) Has been skipped
PR Review / forgejo-review-approach-luna-2 generator (pull_request_target) Has been skipped
PR Review / forgejo-review-approach-luna-3 generator (pull_request_target) Has been skipped
PR Review / forgejo-review-code-smart-1 generator (pull_request_target) Has been skipped
PR Review / forgejo-review-code-luna generator (pull_request_target) Has been skipped
PR Review / forgejo-review-code-luna-2 generator (pull_request_target) Has been skipped
PR Review / forgejo-review-code-luna-3 generator (pull_request_target) Has been skipped
PR Review / Dispatch and observe exact review writers (pull_request_target) Has been skipped
PR Review / Request trusted main review (pull_request_target) Successful in 6s
commit-msg / commitlint (pull_request) Successful in 16s
Checks / quality-checks (26.5.0) (pull_request) Successful in 33s
Checks / quality-checks (24.15.0) (pull_request) Successful in 1m16s
52bf7f9ee6
forgejo-actions left a comment

Approach review: The approach looks good. Replacing the live Router-major lookup with the release manifest's exact axrun pin removes unnecessary runtime coupling and consistently aligns every review lane with the activation contract.

Approach review by Codex CLI · Personal 02 (gpt-5.6-sol)

**Approach review:** The approach looks good. Replacing the live Router-major lookup with the release manifest's exact axrun pin removes unnecessary runtime coupling and consistently aligns every review lane with the activation contract. _Approach review by Codex CLI · Personal 02 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1hcHByb2FjaC1zbWFydC0xIiwic291cmNlV29ya2Zsb3dSdW5JZCI6IjM1ODcwIiwic291cmNlR2VuZXJhdG9yQXR0ZW1wdCI6IjEiLCJyZXN1bHRJZCI6ImFmNzc2OGRjLWIyMDUtNDcyMy04ZGU0LTNlMTYwNThmZWMzMyJ9 -->
forgejo-actions left a comment

Approach review: The approach looks good. Replacing the live Router-major lookup with the release-manifest-pinned axrun version removes an unnecessary network-dependent compatibility layer, and applying the same literal pin to every review lane matches the coordinated activation described in the PR.

Approach review by Codex CLI · Personal 01 (gpt-5.6-sol)

**Approach review:** The approach looks good. Replacing the live Router-major lookup with the release-manifest-pinned axrun version removes an unnecessary network-dependent compatibility layer, and applying the same literal pin to every review lane matches the coordinated activation described in the PR. _Approach review by Codex CLI · Personal 01 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1hcHByb2FjaC1sdW5hLTMiLCJzb3VyY2VXb3JrZmxvd1J1bklkIjoiMzU4NzAiLCJzb3VyY2VHZW5lcmF0b3JBdHRlbXB0IjoiMSIsInJlc3VsdElkIjoiYTQyMmEwYjUtOWQ2ZC00ZTU2LWExMzUtNzUyYmFjODZmYmRhIn0= -->
forgejo-actions left a comment

Approach review: The approach looks good. Replacing live Router-major discovery with the release-manifest-pinned @j4k/axrun@18.0.0 removes unnecessary network-coupled version selection and keeps every review lane on the same activation contract.

Approach review by Codex CLI · Personal 02 (gpt-5.6-sol)

**Approach review:** The approach looks good. Replacing live Router-major discovery with the release-manifest-pinned `@j4k/axrun@18.0.0` removes unnecessary network-coupled version selection and keeps every review lane on the same activation contract. _Approach review by Codex CLI · Personal 02 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1hcHByb2FjaC1sdW5hLTEiLCJzb3VyY2VXb3JrZmxvd1J1bklkIjoiMzU4NzAiLCJzb3VyY2VHZW5lcmF0b3JBdHRlbXB0IjoiMSIsInJlc3VsdElkIjoiODFhZmE2ZWEtOTNlYS00NGZkLWIzNmItZGFhNjQ0YjIxMTkzIn0= -->
forgejo-actions left a comment

Approach review: The approach looks good. Replacing live Router-major discovery with the release manifest's exact axrun pin makes the workflow deterministic and substantially simpler, and applying the same pin across every review lane matches the stated activation process.

Approach review by Codex CLI · Personal 01 (gpt-5.6-sol)

**Approach review:** The approach looks good. Replacing live Router-major discovery with the release manifest's exact axrun pin makes the workflow deterministic and substantially simpler, and applying the same pin across every review lane matches the stated activation process. _Approach review by Codex CLI · Personal 01 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1hcHByb2FjaC1sdW5hLTIiLCJzb3VyY2VXb3JrZmxvd1J1bklkIjoiMzU4NzAiLCJzb3VyY2VHZW5lcmF0b3JBdHRlbXB0IjoiMSIsInJlc3VsdElkIjoiMzI2NGJmZTQtNjlkNC00NDIxLWI1MTQtNzIyM2ZlYWJhNTA0In0= -->
forgejo-actions left a comment

Summary: No actionable issues found.

Code review by Codex CLI · Personal 01 (gpt-5.6-sol)

**Summary:** No actionable issues found. _Code review by Codex CLI · Personal 01 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1jb2RlLXNtYXJ0LTEiLCJzb3VyY2VXb3JrZmxvd1J1bklkIjoiMzU4NzAiLCJzb3VyY2VHZW5lcmF0b3JBdHRlbXB0IjoiMSIsInJlc3VsdElkIjoiOTEyMWZiZTktOWMwOC00ZTliLWE4ODMtMGQ5M2YwZDczMWQ4In0= -->
forgejo-actions left a comment

Summary: No actionable issues found.

Code review by Codex CLI · Personal 02 (gpt-5.6-sol)

**Summary:** No actionable issues found. _Code review by Codex CLI · Personal 02 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1jb2RlLWx1bmEiLCJzb3VyY2VXb3JrZmxvd1J1bklkIjoiMzU4NzAiLCJzb3VyY2VHZW5lcmF0b3JBdHRlbXB0IjoiMSIsInJlc3VsdElkIjoiYzM1NTdkNzUtNzUzZS00ODMzLTlkN2YtMjZlYzJiYTcxOTEyIn0= -->
forgejo-actions left a comment

Summary: No actionable issues found. The activation-gated workflow consistently pins all eight review generators to @j4k/axrun@18.0.0 and removes the obsolete live Router mapping without leaving stale consumers.

Code review by Codex CLI · Personal 01 (gpt-5.6-sol)

**Summary:** No actionable issues found. The activation-gated workflow consistently pins all eight review generators to `@j4k/axrun@18.0.0` and removes the obsolete live Router mapping without leaving stale consumers. _Code review by Codex CLI · Personal 01 (gpt-5.6-sol)_ <!-- axrecipe-review:v2:eyJzY2hlbWFWZXJzaW9uIjoyLCJzdGF0ZSI6InB1Ymxpc2hlZCIsInJlcG9zaXRvcnkiOiJqNGstb3NzL3Ryb3Brb2QtY2xpZW50IiwibnVtYmVyIjoiMTQiLCJiYXNlUmVmIjoibWFpbiIsIm1lcmdlQmFzZVNoYSI6ImIwMDU2MTMxMTMyODBiZjhjN2VmNTlkNTM5NDM5MmU4NzNlNjRhNjYiLCJoZWFkU2hhIjoiNTJiZjdmOWVlNjdhMjViY2RlMzM4ZTdmNTBlYTU4MDJkMjQ3MzllNSIsInNsb3QiOiJmb3JnZWpvLXJldmlldy1jb2RlLWx1bmEtMyIsInNvdXJjZVdvcmtmbG93UnVuSWQiOiIzNTg3MCIsInNvdXJjZUdlbmVyYXRvckF0dGVtcHQiOiIxIiwicmVzdWx0SWQiOiJmYjZiNTA4OS05MTIxLTRhYWQtYmZkZi1jZTgzZjVlMjA0ZTIifQ== -->
jercik merged commit bda1fbf57d into main 2026-09-01 09:06:55 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
j4k-oss/tropkod-client!14
No description provided.