Skip to content

fix(release): require observed config publication - #451

Merged
AprilNEA merged 2 commits into
masterfrom
xuan/config-observer-release-gate
Aug 15, 2026
Merged

fix(release): require observed config publication#451
AprilNEA merged 2 commits into
masterfrom
xuan/config-observer-release-gate

Conversation

@AprilNEA

Copy link
Copy Markdown
Member

Summary

  • accept release-manifest v2 and strictly validate its Pages observer evidence
  • require v2 evidence for production and all real build plans while retaining v1 only for the reviewed nonproduction fixture
  • bind deployment source to sourceGitSha and require one deployment/source/key/verification identity across platform manifests
  • preserve independent configurable publisher/source repository checkouts and existing scoped App tokens

Validation

  • focused release workflow/render tests (32/32)
  • pnpm check:ci
  • pnpm test (337 files passed, 3 skipped; 2916 tests passed, 5 skipped)
  • workflow YAML parse
  • git diff --check

This PR does not publish a release or change GitHub Environment variables or secrets.

Copilot AI lite review requested due to automatic review settings August 15, 2026 12:19
@pullfrog

pullfrog Bot commented Aug 15, 2026

Copy link
Copy Markdown

Your Claude subscription has hit its usage limit. It resets at 9pm (UTC). Re-trigger Pullfrog after the reset, or add an ANTHROPIC_API_KEY repo secret — Pullfrog routes around an exhausted subscription automatically when one is present.

Add repo secret → · Model settings → · Setup docs → · Ask in Discord →

Pullfrog  | Rerun failed job ➔View workflow run | via Pullfrog | Using Claude Opus𝕏

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@AprilNEA
AprilNEA requested a review from Yidadaa August 15, 2026 12:19

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5f8ed5e714

ℹ️ 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".

Comment thread .github/actions/render-release-config/action.yml
Copilot AI review requested due to automatic review settings August 15, 2026 12:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@pullfrog

pullfrog Bot commented Aug 15, 2026

Copy link
Copy Markdown

Your Claude subscription has hit its usage limit. It resets at 9pm (UTC). Re-trigger Pullfrog after the reset, or add an ANTHROPIC_API_KEY repo secret — Pullfrog routes around an exhausted subscription automatically when one is present.

Add repo secret → · Model settings → · Setup docs → · Ask in Discord →

Pullfrog  | Rerun failed job ➔View workflow run | via Pullfrog | Using Claude Opus𝕏

@AprilNEA
AprilNEA merged commit 7677211 into master Aug 15, 2026
11 of 12 checks passed
@AprilNEA
AprilNEA deleted the xuan/config-observer-release-gate branch August 15, 2026 13:16
@linear-code

linear-code Bot commented Aug 15, 2026

Copy link
Copy Markdown

CORE-125

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.

2 participants