PR_26179_OWNER_012-project-instructions-startup-validation #257
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| # Project Instructions Version | ||
|
|
||
| Current Project Instructions Version: 2026.06.28.001 | ||
|
|
||
| Last Updated: 2026-06-28 | ||
|
|
||
| ## Breaking Changes Summary | ||
|
|
||
| - Codex startup validation is mandatory before every task outcome. | ||
| - Project Instructions versions are repository-owned and increment independently of PR numbers. | ||
| - Codex must read this version file and the latest repository copy of `PROJECT_INSTRUCTIONS.md` before performing work. | ||
| - Codex must discard previously remembered Project Instructions and treat the repository copy as authoritative. | ||
| - Codex must validate canonical report and ZIP paths, branching model, and legacy path avoidance before work proceeds. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| # PR_26179_OWNER_012-project-instructions-startup-validation Branch Validation | ||
|
|
||
| ## Result | ||
|
|
||
| PASS | ||
|
|
||
| ## Checks | ||
|
|
||
| | Check | Result | Notes | | ||
| | --- | --- | --- | | ||
| | Stacked PR model identified | PASS | This PR depends on PR #256. | | ||
| | Started from documented previous PR branch | PASS | Started from `PR_26179_OWNER_011-team-stacked-pr-policy`. | | ||
| | Worktree clean before branch creation | PASS | Verified before branch creation. | | ||
| | PR branch created | PASS | `PR_26179_OWNER_012-project-instructions-startup-validation`. | | ||
| | Work stayed on PR branch | PASS | No direct commit to `main`. | | ||
| | Documentation/governance scope | PASS | Only Project Instructions and reports changed. | |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| # PR_26179_OWNER_012-project-instructions-startup-validation Manual Validation Notes | ||
|
|
||
| ## Notes | ||
|
|
||
| - Confirmed the Startup Contract is the first section in `PROJECT_INSTRUCTIONS.md`. | ||
| - Confirmed `PROJECT_INSTRUCTIONS_VERSION.md` exists in the active Project Instructions folder. | ||
| - Confirmed Project Instructions version uses repository-owned `2026.06.28.001` format rather than PR-number format. | ||
| - Confirmed the startup response block includes `Instruction Source`, `Repository ........ PASS`, and `Cached Memory ..... DISCARDED`. | ||
| - Confirmed the Startup Contract says conversation memory must never override repository instructions. | ||
| - Confirmed the startup response block uses `dev/reports` and `dev/workspace/zips`. | ||
| - Confirmed the legacy path check calls out `tmp/` and `docs_build/` as not used. | ||
| - Confirmed no runtime or product files were modified. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| # PR_26179_OWNER_012-project-instructions-startup-validation Report | ||
|
|
||
| ## Purpose | ||
|
|
||
| Strengthen the Project Instructions startup contract so every Codex task proves it loaded the latest Project Instructions before performing work. | ||
|
|
||
| ## Branching Model | ||
|
|
||
| - Model: Stacked PR | ||
| - Previous PR dependency: `PR_26179_OWNER_011-team-stacked-pr-policy` / PR #256 | ||
| - Starting branch: `PR_26179_OWNER_011-team-stacked-pr-policy` | ||
| - Merge order: PR #255, then PR #256, then this PR | ||
|
|
||
| ## Governance Outcome | ||
|
|
||
| - Added `dev/build/ProjectInstructions/PROJECT_INSTRUCTIONS_VERSION.md`. | ||
| - Replaced PR-shaped Project Instructions versioning with repository-owned version `2026.06.28.001`. | ||
| - Added a mandatory Startup Contract at the top of `PROJECT_INSTRUCTIONS.md`. | ||
| - Required Codex to read the version file and latest repository Project Instructions before every task. | ||
| - Required Codex to discard remembered Project Instructions in favor of the repository copy. | ||
| - Added explicit startup output for `Instruction Source`, proving repository instructions are used and cached memory is discarded. | ||
| - Documented that conversation memory must never override repository instructions. | ||
| - Required startup validation for canonical paths, branching model, ZIP/report locations, and legacy path avoidance. | ||
| - Added failure behavior when the version or canonical paths cannot be determined. | ||
| - Documented that the startup validation applies to PLAN, BUILD, APPLY, Review, Audit, Governance, Validation, Read-only, Hard stop, No-op, and Partial completion outcomes. | ||
|
|
||
| ## Runtime Impact | ||
|
|
||
| None. This PR changes documentation/governance only. | ||
|
|
||
| ## ZIP | ||
|
|
||
| - `dev/workspace/zips/PR_26179_OWNER_012-project-instructions-startup-validation_delta.zip` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,19 @@ | ||
| # PR_26179_OWNER_012-project-instructions-startup-validation Requirement Checklist | ||
|
|
||
| | Requirement | Result | Notes | | ||
| | --- | --- | --- | | ||
| | Create `PROJECT_INSTRUCTIONS_VERSION.md` | PASS | Added under `dev/build/ProjectInstructions/`. | | ||
| | Version file contains current version | PASS | `2026.06.28.001`. | | ||
| | Version format is repository-owned, not PR-owned | PASS | Version increments independently of PR numbers. | | ||
| | Version file contains last updated | PASS | `2026-06-28`. | | ||
| | Version file contains breaking changes summary | PASS | Added concise startup validation summary. | | ||
| | Add mandatory Startup Contract at top of `PROJECT_INSTRUCTIONS.md` | PASS | Added before the main Project Instructions heading. | | ||
| | Require reading version file and latest Project Instructions | PASS | Included in Startup Contract. | | ||
| | Require repository copy as authoritative and discard memory | PASS | Included in Startup Contract. | | ||
| | Conversation memory must never override repository instructions | PASS | Included in Startup Contract. | | ||
| | Require loading referenced instruction documents | PASS | Included in Startup Contract. | | ||
| | Require canonical path, branching model, ZIP/report validation | PASS | Included in Startup Contract. | | ||
| | Add mandatory response block | PASS | Added exact startup validation response format with Instruction Source section. | | ||
| | Add failure behavior | PASS | Added STOP example and pre-work stop rule. | | ||
| | Apply to every Codex task type | PASS | Listed all requested task/outcome types. | | ||
| | Do not modify implementation code | PASS | Documentation/governance only. | |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| # PR_26179_OWNER_012-project-instructions-startup-validation Validation Report | ||
|
|
||
| ## Validation | ||
|
|
||
| - `git diff --check`: PASS | ||
| - `npm run validate:canonical-structure`: PASS | ||
| - Project Instructions grep for startup validation contract: PASS | ||
| - Project Instructions grep for repository-owned version format and Instruction Source output: PASS | ||
|
|
||
| ## Playwright | ||
|
|
||
| Not impacted. No runtime, UI, API, database, or product page files changed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For review or validation runs where the caller requires a strict JSON or other machine-readable payload, this unconditional prefix makes it impossible to satisfy the caller while also following Project Instructions, and the new section explicitly applies to Review and Validation outcomes. In those contexts the startup prose corrupts the response before the required payload can be parsed, so the contract needs an exception or alternate artifact-based proof for schema-constrained outputs.
Useful? React with 👍 / 👎.