Skip to content

fix(coverage): finish legacy Actions retirement - #728

Merged
devantler merged 1 commit into
mainfrom
codex/final-actions-retirement-240-r56
Oct 23, 2026
Merged

devantler merged 1 commit into
mainfrom
codex/final-actions-retirement-240-r56

Conversation

@devantler

Copy link
Copy Markdown
Contributor

🤖 Generated by the Agentic Engineer

Why

The retired automation repository has now been deleted after its consumers and protection were migrated. Its temporary exemption would make the next governance audit fail and could conceal an accidental recreation.

What

Remove the obsolete exception while retaining complete-coverage and failure checks. Update the catalogue and contributor guidance to identify the maintained automation and preserve historical verification and recovery context.

Fixes #240

@github-code-quality

github-code-quality Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Code coverage overview

Languages: Go, C#

Go / code-coverage/test

main 93a7889 codex/final-actions-... 29f95fd +/-
50% 50% 0%

C# / code-coverage/dotnet

main 93a7889 codex/final-actions-... 29f95fd +/-
100% 100% 0%

Go / code-coverage/go

main 93a7889 codex/final-actions-... 29f95fd +/-
50% 100% +50%
View coverage of the 1 most impacted file.
File main 93a7889 codex/final-actions-... 29f95fd +/-
value.go 0% 100% +100%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.


Updated October 23, 2026 10:18 UTC

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Agentic Engineer

The deletion and independent absence evidence are recorded on #240. This follow-up changes no managed resource or credential boundary.

TDD: the post-deletion clean census fixture first failed on the obsolete Actions exemption. After removing it, the actual coverage client passes its complete stable paginated installation controls, while recreated Actions, unseen repositories, incomplete reads, changed installation metadata and a synthetic stale exception still fail for their intended reasons. The extra pagination fixture is an explicitly archived synthetic repository, not a hidden production exception.

All 15 repository Maintenance baseline commands pass, including rendered withdrawal and complete coverage. Instruction layout passes its 20 negative controls; README interface parity passes all 43 behavioral cases; ShellCheck with source following and diff checks pass. Both action/workflow indexes retain the same parallel collapsible structure and all existing interfaces.

Fresh native CI, current-head substantive review, protected merge and the normal reviewed-main live governance audit remain required. No historic signer identity or archived evidence is rewritten.

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Agentic Engineer

Review hand-off at 29f95fdae6bb7790447852b9c0c6401dbe2fff60: complete native CI is green. The prior review route remains unavailable; current-head artifact guards report ADMIT, so progression continues to Codex.

This is the final post-deletion correction, not a deletion request. Verified authenticated absence and recovery evidence are linked from #240. The temporary exception is removed; the complete actual-client controls still reject unseen or recreated repositories, partial/unstable inventory, inappropriate installation metadata and a synthetic stale exception. Documentation reflects the maintained catalogue and preserves historical identity data. No managed resource or credential grant changes.

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Agentic Engineer

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-23T10:51:00.623389Z 29f95fd Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 29f95fdae6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Agentic Engineer

Ready at signed head 29f95fdae6bb7790447852b9c0c6401dbe2fff60, based on freshly fetched main 93a788988d6c9e5b96594b07d4919a53f8b61975. Complete native CI is green; the current-head Codex review and companion summary are finding-free, and the complete thread inventory is empty. The published body follows the repository template.

All 15 local baseline commands, complete coverage-client controls, 43 README parity cases, 20 instruction-layout mutations, ShellCheck with source following and diff checks pass. The post-deletion clean fixture first failed on the stale exemption. Recreated Actions, unseen repositories, unstable/partial reads and a synthetic stale exception remain blocking; no managed resource, credential boundary, historical identity or required check is removed.

Repository deletion, positive authentication controls, complete organization absence and the verified private recovery backup are recorded on #240. Actions and reusable-workflow references retain the same parallel indexes and collapsibles. Promotion will retain any new native checks; protected merge, normal postmerge validation and the reviewed-main live governance audit remain required before final delivery is recorded.

@devantler
devantler marked this pull request as ready for review October 23, 2026 11:03
@devantler
devantler merged commit 3bc75c3 into main Oct 23, 2026
272 checks passed
@devantler
devantler deleted the codex/final-actions-retirement-240-r56 branch October 23, 2026 11:07
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.

Fully delete devantler-tech/actions after verified consumer migration

1 participant