Skip to content

ci: remove unused changed-file workflow boundary - #79

Draft
Jeremy1844 wants to merge 2 commits into
Adventech:stagefrom
Jeremy1844:fix/adv-2026-0044-resources-change-boundary-stage-9d237dd4
Draft

ci: remove unused changed-file workflow boundary#79
Jeremy1844 wants to merge 2 commits into
Adventech:stagefrom
Jeremy1844:fix/adv-2026-0044-resources-change-boundary-stage-9d237dd4

Conversation

@Jeremy1844

Copy link
Copy Markdown

Root cause

Four resource deployment workflows run the changed-files action twice, but no later step or job consumes either invocation's outputs. That leaves an unnecessary repository-filename processing boundary in deployment automation. These workflows already deploy their complete generated output for each matching push, so removing the unused steps preserves the intended deployment scope.

Change

  • remove the two unused changed-file invocations and JSON-output setup from each affected workflow
  • add a dependency-free regression that keeps all four deployment jobs in scope and rejects reintroduction of the unused boundary

Evidence

  • base: 9d237dd4fb37dd1fb936e3f9649020cc43a374c8 (stage)
  • regression: 0329cd8080946e12bc4461cfd576a9bf08c2dd82 — 0/4 introduced cases passed
  • fix: 16864ea8626807407bea6181992fd339a111aaf0 — 4/4 introduced cases passed
  • all eight repository workflows parse with exact js-yaml 4.1.0
  • git diff --check, exact five-path scope, workflow-policy, protected-tree, and added-secret-pattern checks pass
  • src, assets, package.json, and yarn.lock objects are unchanged from the base
  • exact patch IDs match the prior audit proof while both new commits use the public noreply identity

Open-PR composition

These are separate root causes and should remain separate pull requests.

Rollout and recovery

This changes no content, media, package, lockfile, schema, stored artifact, or user data, so no migration is required. Run secret-free pull-request policy checks before merge; do not use a stage or production deployment as the test harness. Historical workflow, token-permission, and cloud-access review remains a separate maintainer-owned task.

If deployment must be paused or another workflow change rolled back, retain this deletion/guard or disable the automation. Do not restore the unused action boundary as a rollback.

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