Skip to content

refactor(ci)!: make release inspection a manual repository workflow - #697

Merged
devantler merged 2 commits into
mainfrom
codex/manual-inspector-ownership-r29
Oct 21, 2026
Merged

devantler merged 2 commits into
mainfrom
codex/manual-inspector-ownership-r29

Conversation

@devantler

Copy link
Copy Markdown
Contributor

🤖 Generated by the Agentic Engineer

Why

Release inspection is a manual diagnostic owned by this repository, but its unused reusable interface makes it look like a shared product. The only caller exists to exercise that unused interface, rather than serving a production consumer.

What

Keep the default-off manual diagnostic and retire its reusable interface and artificial caller. Preserve read-only authorization, source protection, real diagnostic coverage and clear operator guidance.

Fixes #695

💥 Breaking change: Release inspection can no longer be called by other workflows; run it manually in this repository. Existing immutable release references remain available.

BREAKING CHANGE: Remove the unused reusable inspection interface. Run the default-off manual workflow in the catalogue repository instead. Existing immutable release pins remain available.
@github-code-quality

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

Copy link
Copy Markdown

Code coverage overview

Languages: Go, C#

Go / code-coverage/test

main 335a716 codex/manual-inspect... 7b6d7cf +/-
50% 50% 0%

C# / code-coverage/dotnet

main 335a716 codex/manual-inspect... 7b6d7cf +/-
100% 100% 0%

Go / code-coverage/go

main 335a716 codex/manual-inspect... 7b6d7cf +/-
50% 50% 0%

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


Updated October 21, 2026 22:44 UTC

@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Agentic Engineer

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 21, 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-21T22:29:28.391257Z 7b6d7cf 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. Swish!

Reviewed commit: 7b6d7cf9be

ℹ️ 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
devantler marked this pull request as ready for review October 21, 2026 22:48
@devantler
devantler merged commit 8e43018 into main Oct 21, 2026
499 of 520 checks passed
@devantler
devantler deleted the codex/manual-inspector-ownership-r29 branch October 21, 2026 22:48
@devantler

Copy link
Copy Markdown
Contributor Author

🤖 Generated by the Agentic Engineer

Merged as 8e43018 after current-head Codex review and 213 successful checks with 60 intended skips. Published v9.0.0; Release 39689158043 and CD 39689217767 succeeded.

Native reviewed-main manual dispatch 39689236183 is bound to that merge commit. With enable-inspection=false, its sole inspection job was skipped with zero executed steps: no diagnostic or credential issuance occurred. Enabled diagnostic behavior remains covered by the existing real HTTP and source/authority controls; this disabled smoke is not an enabled diagnosis or publication-recovery claim. The unused reusable interface and artificial caller are retired; historical immutable releases remain available. Issue #695 is verified.

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.

Own release inspection as a manual repository workflow

1 participant