Skip to content

fix(ci): restore normal image publishing on main - #131

Merged
aoirint merged 1 commit into
mainfrom
codex/remove-apm-publication-guard
Oct 28, 2026
Merged

aoirint merged 1 commit into
mainfrom
codex/remove-apm-publication-guard

Conversation

@aoirint

@aoirint aoirint commented Oct 28, 2026

Copy link
Copy Markdown
Owner

Warning

This pull request was created with assistance from LLMs.

Summary

Remove the application-change classifier introduced in the APM rollout. Restore the existing main-push workflow: source checks and release planning gate the image build/publication path, including APM-only changes. Keep the existing release identity, publication permissions, tests and Release condition.

The maintainer explicitly approved automatic Docker image publication from this merge. The Skill deployment and APM 0.34.0 selection remain unchanged.

Related Issues

Notes for reviewers

Delete .github/scripts/application-change-scope.sh, its Plan step/output and the Build condition, then remove the associated guide text. The resulting main workflow matches the pre-classifier workflow. No new trigger, action pin, permission, image variant or VERSION change is introduced. The existing VERSION tag and immutable release are present, so the current expected path is edge image publication and a skipped GitHub Release job. Required head CI and integrated-main results will be checked separately; no manual publication or extra tag is requested.

Proposed merge attribution

  • Included: Co-authored-by: Codex <noreply@openai.com> — implementation and verification. The PR creator remains primary author; no additional human co-author is proposed.

AI disclosure

Codex prepared the scoped removal and validation. An independent AI reviewer inspected the final workflow, guide diff, preserved APM files and publication conditions. No human manual testing is claimed.

Testing

Automated checks

  • Main workflow byte/structure comparison with its pre-classifier version, unchanged APM manifest/lock/notices/0.34 action and generated Skills, no stale classifier references, and git diff --check: passed.
  • Actionlint with available ShellCheck integration, pinact run --check --verify --min-age 7, and repository-configured Markdown lint: passed. No changed standalone shell script remains; the classifier is deleted.
  • Container builds and publication were not run locally. The existing head checks and automatic main build/publication path remain required.

AI-assisted inspections

  • Request: Restore the original publishing path without changing Skill deployment, release identity or validation gates.
    • AI-assisted result: The diff is limited to classifier removal and its associated workflow/documentation; existing trust boundaries and release rules are preserved.

Restore the original main-push image publishing path; preserve the APM deployment.

Co-authored-by: Codex <noreply@openai.com>
@aoirint
aoirint marked this pull request as ready for review October 28, 2026 00:43
@aoirint
aoirint merged commit 88fcb4a into main Oct 28, 2026
1 check passed
@aoirint
aoirint deleted the codex/remove-apm-publication-guard branch October 28, 2026 00:43
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