Skip to content

ci: adopt advisory tool-attribution checks - #602

Merged
ss-o merged 1 commit into
mainfrom
feature-764
Oct 8, 2026
Merged

ss-o merged 1 commit into
mainfrom
feature-764

Conversation

@ss-o

@ss-o ss-o commented Oct 8, 2026

Copy link
Copy Markdown
Member

The current workflow pin rejects recognized automatic tool credits. Update it to the signed revision in z-shell/.github#764, where these produce advisory warnings. Managed agents still omit automatic credits.

Only the pin and its policy comment change. Permissions, triggers, caller inputs, and the commit, title, branch and issue checks are preserved.

Refs z-shell/.github#652. Depends on z-shell/.github#764 before merge. The rationale and evidence live in that policy change.

Validation: Actionlint, YAML formatting and whitespace checks; the shared workflow's 58 behavior checks pass, including attributed commits succeeding and invalid subjects failing. Application source is unchanged.

Pin the shared workflow to the attribution-preference change.
Preserve the caller inputs and all other policy checks.

Refs: z-shell/.github#652

@ss-o ss-o left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fallback review under ADR-0026: maintainer elected, no Copilot request on 40f06c2

No actionable introduced defects found. Reviewed the complete PR diff against the repository contract and organization criteria.

Workflow and supply chain: pass. The caller pins signed published revision 64a427d8ac33c9ef1dc89af35920eb0d3de9fa9c. Inspection of the old-to-new shared workflow confirms advisory attribution diagnostics while preserving commit, title, branch and issue checks. Its source is byte-identical to the updated organization PR head.

Scope and compatibility: pass. Permissions, event triggers, concurrency ownership and caller inputs are preserved. The policy comment describes warning behavior. No application source changed.

Validation: pass. Actionlint, YAML formatting and whitespace checks passed, and the shared actual-workflow suite passed 58 checks. Current hosted checks are green. Native commit hooks passed without bypass.

Limits: Self-review, not independent approval. Merge depends on adoption of organization PR764. Local application suites were outside this pin-only change; hosted results provide the available integration evidence.

@ss-o
ss-o merged commit acb52e8 into main Oct 8, 2026
15 checks passed
@ss-o
ss-o deleted the feature-764 branch October 8, 2026 12:38
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