Skip to content

Prepare open-scaffold@0.34.0 release sync - #240

Merged
graphanov merged 2 commits into
mainfrom
forge/issue-239-release-open-scaffold-ambient-capture-trust-package
Jun 28, 2026
Merged

Prepare open-scaffold@0.34.0 release sync#240
graphanov merged 2 commits into
mainfrom
forge/issue-239-release-open-scaffold-ambient-capture-trust-package

Conversation

@graphanov

Copy link
Copy Markdown
Owner

Summary

  • Prepare the open-scaffold@0.34.0 release-sync candidate from current main without performing any release action.

Scope

  • Bumps package metadata to 0.34.0, adds the v0.34.0 changelog candidate entry, records the repo-native plan/evidence note, and refreshes the live-corpus hash test for those records.

Out of scope

  • No merge, npm publish, workflow dispatch, tag, GitHub Release, branch protection, settings, secrets, or post-publish smoke was performed.

Verification

  • ./verify.sh --strict → 10 pass / 0 fail / 0 warn; npm test -- --run → 51 files / 625 tests passed; npm run build → passed; npm pack --dry-run --json → open-scaffold@0.34.0, 228 files.

Risk

Linked issue

Refs #239 — keep open for owner-gated npm publish, tag/GitHub Release, and post-publish proof after review/merge.

Authority boundary

Draft PR only; no merge, publish, release, tag, workflow dispatch, settings, branch protection, force-push, or secrets authority used.

@graphanov

Copy link
Copy Markdown
Owner Author

Status: blocked — draft promotion is blocked by release-gate path changes.

Evidence:

  • PR Prepare open-scaffold@0.34.0 release sync #240 head 52be6fb is mergeable/CLEAN with CI pass, zero review threads, and local verification passing: ./verify.sh --strict && npm test -- --run; npm run build. Diff includes package.json:version, package-lock.json:version, and .osc/releases/**, which are configured forbidden paths.

Needed: Explicit owner decision to move this release-sync PR out of draft and request independent review despite those forbidden-path changes. Merge, publish, release, tag, and workflow-dispatch remain separate owner gates.

@graphanov

Copy link
Copy Markdown
Owner Author

Status: blocked — draft promotion and Codex review are owner-gated by configured release-path changes.

Evidence:

Needed: Owner approval with exact scope, for example: APPROVE JOHN-LOMEIN PR #240 DRAFT PROMOTION: mark PR #240 ready and request Codex review for head 52be6fb despite package version and .osc/releases path changes; do not merge, publish, tag, create a GitHub Release, dispatch workflows, force-push, change settings, or touch secrets.

@graphanov

Copy link
Copy Markdown
Owner Author

APPROVE JOHN-LOMEIN PR #240 DRAFT PROMOTION: mark PR #240 ready and request Codex review for head 52be6fb despite package version and .osc/releases path changes; do not merge, publish, tag, create a GitHub Release, dispatch workflows, force-push, change settings, or touch secrets.

@graphanov
graphanov marked this pull request as ready for review June 28, 2026 15:57
@graphanov

Copy link
Copy Markdown
Owner Author

Status: ready for review on approved draft-promotion gate.

Evidence:

  • Owner approval comment authorizes draft promotion and Codex review for head 52be6fb; merge/publish/release/tag/workflow-dispatch remain excluded.
    GitHub: MERGEABLE/CLEAN, CI success, zero review threads.
    Local: ./verify.sh --quick --quiet; git diff --check origin/main...HEAD; ./verify.sh --strict && npm test -- --run; npm run build -> passed.

Next: Requesting one Codex review for the current head; no merge, publish, release, tag, workflow dispatch, force-push, settings, or secrets action taken.

@graphanov

Copy link
Copy Markdown
Owner Author

@codex review

@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: 52be6fb70c

ℹ️ 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 package.json
@graphanov

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 442924f8e7

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

@graphanov
graphanov merged commit 9fe5f1a into main Jun 28, 2026
6 checks passed
@graphanov
graphanov deleted the forge/issue-239-release-open-scaffold-ambient-capture-trust-package branch June 28, 2026 18:52
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