Skip to content

Commit 78827f1

Browse files
authored
Merge pull request #246 from ToolboxAid/PR_26179_OWNER_012-gitignore-env-simplification
PR_26179_OWNER_012-gitignore-env-simplification
2 parents c307ab4 + 7f808be commit 78827f1

6 files changed

Lines changed: 118 additions & 107 deletions

.gitignore

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,8 @@ docs/dev/reports/playwright_v8_coverage_report.txt
4848
*.mp4
4949
*.mkv
5050

51-
.env
52-
.env.*
51+
# Environment files
52+
.env*
5353
!.env.example
5454
!.env.sample
5555
!.env.template
Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
# PR_26179_OWNER_012-gitignore-env-simplification
2+
3+
Updated: 2026-06-28T03:47:16Z
4+
Team: Owner
5+
Branch: PR_26179_OWNER_012-gitignore-env-simplification
6+
Scope: .gitignore only plus required reports and outcome ZIP. No runtime code or production pages changed.
7+
8+
## Summary
9+
- Replaced the two-line environment ignore rule with a single `.env*` rule.
10+
- Preserved exceptions for `.env.example`, `.env.sample`, and `.env.template`.
11+
12+
## Changed Files
13+
```text
14+
M .gitignore
15+
M dev/reports/codex_changed_files.txt
16+
M dev/reports/codex_review.diff
17+
A dev/reports/PR_26179_OWNER_012-gitignore-env-simplification.md
18+
A dev/reports/PR_26179_OWNER_012-gitignore-env-simplification_requirement-checklist.md
19+
A dev/reports/PR_26179_OWNER_012-gitignore-env-simplification_validation-report.md
20+
```
21+
22+
## Validation Matrix
23+
| Path | Result | Status |
24+
| --- | --- | --- |
25+
| .env | ignored | PASS |
26+
| .env.dev | ignored | PASS |
27+
| .env.ist | ignored | PASS |
28+
| .env.uat | ignored | PASS |
29+
| .env.prd | ignored | PASS |
30+
| .env.local | ignored | PASS |
31+
| .env.example | tracked and not ignored | PASS |
32+
| .env.sample | not ignored by exception; file not present/tracked in repo | PASS_WITH_SCOPE_NOTE |
33+
| .env.template | not ignored by exception; file not present/tracked in repo | PASS_WITH_SCOPE_NOTE |
34+
35+
Scope note: `.env.sample` and `.env.template` are not currently tracked files in this repository, so the validation confirms the requested exception behavior without adding files outside the .gitignore-only scope.
36+
37+
## Validation
38+
- `git diff --check`: PASS.
39+
- Environment ignore matrix: PASS with scope note for absent sample/template files.
40+
41+
## Blockers
42+
None.
Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
# PR_26179_OWNER_012-gitignore-env-simplification Requirement Checklist
2+
3+
Updated: 2026-06-28T03:47:16Z
4+
5+
- [x] Current branch started from main.
6+
- [x] Worktree was clean before changes.
7+
- [x] Updated `.gitignore` only for the requested source change.
8+
- [x] Replaced `.env` and `.env.*` with `.env*`.
9+
- [x] Added `# Environment files` comment.
10+
- [x] Preserved `!.env.example`.
11+
- [x] Preserved `!.env.sample`.
12+
- [x] Preserved `!.env.template`.
13+
- [x] `.env` ignored.
14+
- [x] `.env.dev` ignored.
15+
- [x] `.env.ist` ignored.
16+
- [x] `.env.uat` ignored.
17+
- [x] `.env.prd` ignored.
18+
- [x] `.env.local` ignored.
19+
- [x] `.env.example` tracked and not ignored.
20+
- [x] `.env.sample` exception active; file is absent/not tracked in repo.
21+
- [x] `.env.template` exception active; file is absent/not tracked in repo.
22+
- [x] `git diff --check` passed.
Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
# PR_26179_OWNER_012-gitignore-env-simplification Validation Report
2+
3+
Updated: 2026-06-28T03:47:16Z
4+
5+
## Commands
6+
- `git diff --check`: PASS
7+
- `git check-ignore` env matrix: PASS
8+
- `git ls-files --error-unmatch .env.example`: PASS
9+
10+
## Env Matrix
11+
| Path | Result | Status |
12+
| --- | --- | --- |
13+
| .env | ignored | PASS |
14+
| .env.dev | ignored | PASS |
15+
| .env.ist | ignored | PASS |
16+
| .env.uat | ignored | PASS |
17+
| .env.prd | ignored | PASS |
18+
| .env.local | ignored | PASS |
19+
| .env.example | tracked and not ignored | PASS |
20+
| .env.sample | not ignored by exception; file not present/tracked in repo | PASS_WITH_SCOPE_NOTE |
21+
| .env.template | not ignored by exception; file not present/tracked in repo | PASS_WITH_SCOPE_NOTE |
22+
23+
## Scope Checks
24+
- Runtime code changed: NO
25+
- Production pages changed: NO
26+
- Source change limited to `.gitignore`: YES
27+
28+
## Notes
29+
- `.env.sample` and `.env.template` are unignored by the exception rules, but they are not currently tracked files. They were not added because the PR scope allows source changes to `.gitignore` only.
30+
31+
## Blockers
32+
None.
Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,6 @@
1-
PR_26179_OWNER_010-canonical-project-folder-instructions
2-
Updated: 2026-06-28T02:50:06Z
3-
Branch: PR_26179_OWNER_010-canonical-project-folder-instructions
1+
PR_26179_OWNER_012-gitignore-env-simplification
2+
Updated: 2026-06-28T03:47:16Z
3+
Branch: PR_26179_OWNER_012-gitignore-env-simplification
44

5-
Changed files staged for this update:
6-
M dev/build/ProjectInstructions/addendums/pr_workflow.md
7-
M dev/build/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md
8-
M dev/build/ProjectInstructions/bootstrap/codex_start_of_day_bootstrap.md
9-
M dev/build/ProjectInstructions/standards/CODEX_WORKFLOW_COMMANDS.md
10-
11-
Working tree status at report generation:
12-
M dev/build/ProjectInstructions/addendums/pr_workflow.md
13-
M dev/build/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md
14-
M dev/build/ProjectInstructions/bootstrap/codex_start_of_day_bootstrap.md
15-
M dev/build/ProjectInstructions/standards/CODEX_WORKFLOW_COMMANDS.md
5+
Changed files:
6+
M .gitignore

dev/reports/codex_review.diff

Lines changed: 15 additions & 91 deletions
Original file line numberDiff line numberDiff line change
@@ -1,91 +1,15 @@
1-
diff --git a/dev/build/ProjectInstructions/addendums/pr_workflow.md b/dev/build/ProjectInstructions/addendums/pr_workflow.md
2-
index ce6483838..6fadd0789 100644
3-
--- a/dev/build/ProjectInstructions/addendums/pr_workflow.md
4-
+++ b/dev/build/ProjectInstructions/addendums/pr_workflow.md
5-
@@ -41,7 +41,8 @@ This file owns PR lifecycle governance. It must not duplicate command phase rule
6-
Team-neutral daily workflow:
7-
8-
- SOD starts from the latest synchronized `main`.
9-
-- SOD creates or updates one active team branch/workstream.
10-
+- SOD reports the active team branch/workstream recommendation only.
11-
+- Branch creation or branch updates happen after SOD through the appropriate PR execution phase.
12-
- Work must be committed only to the active team branch.
13-
- PR branches/commits stay on the active team branch/workstream during the day when OWNER assigned a stacked or sequential workstream.
14-
- Do not return to `main` between stacked/sequential PRs in the same active workstream.
15-
diff --git a/dev/build/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md b/dev/build/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md
16-
index e191f9d7f..1e6a04a87 100644
17-
--- a/dev/build/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md
18-
+++ b/dev/build/ProjectInstructions/addendums/team_backlog_sod_eod_standard.md
19-
@@ -88,7 +88,8 @@ At EOD, ChatGPT/Codex must provide:
20-
This standard preserves the existing branch workflow:
21-
22-
- SOD starts from latest `main`.
23-
-- SOD creates or uses the active team branch.
24-
+- SOD reports the recommended active team branch/workstream only.
25-
+- Branch creation or use happens after SOD through the appropriate PR execution phase.
26-
- All commits go to the active team branch, not `main`.
27-
- Sequential PRs stay on the active team branch/workstream during the day.
28-
- EOD merges OWNER-approved work, pushes, returns to `main`, and verifies clean worktree and local/origin sync `0 0`.
29-
diff --git a/dev/build/ProjectInstructions/bootstrap/codex_start_of_day_bootstrap.md b/dev/build/ProjectInstructions/bootstrap/codex_start_of_day_bootstrap.md
30-
index 280abef00..f2eb2787d 100644
31-
--- a/dev/build/ProjectInstructions/bootstrap/codex_start_of_day_bootstrap.md
32-
+++ b/dev/build/ProjectInstructions/bootstrap/codex_start_of_day_bootstrap.md
33-
@@ -33,7 +33,24 @@ The bootstrap loads:
34-
- Current branch
35-
- Worktree status
36-
37-
-The bootstrap must not create branches, commit files, move files, or run implementation work by itself.
38-
+The bootstrap requires the current branch to be `main`, the worktree to be clean, and local `main` to be synced with `origin/main`.
39-
+
40-
+Allowed bootstrap sync actions:
41-
+
42-
+```text
43-
+git fetch origin
44-
+git pull --ff-only origin main
45-
+```
46-
+
47-
+If the current branch is not `main`, the bootstrap must HARD STOP and return:
48-
+
49-
+```text
50-
+git switch main
51-
+git pull --ff-only origin main
52-
+Start of Day {Team}
53-
+```
54-
+
55-
+The bootstrap must not create branches, switch branches, commit files, move files, or run implementation work by itself.
56-
57-
Start-of-Day bootstrap must satisfy the Codex Completion Contract in `dev/build/ProjectInstructions/addendums/codex_artifact_and_reporting_standard.md` with a review-only ZIP unless a hard stop occurs.
58-
59-
diff --git a/dev/build/ProjectInstructions/standards/CODEX_WORKFLOW_COMMANDS.md b/dev/build/ProjectInstructions/standards/CODEX_WORKFLOW_COMMANDS.md
60-
index 25a403486..c0066966e 100644
61-
--- a/dev/build/ProjectInstructions/standards/CODEX_WORKFLOW_COMMANDS.md
62-
+++ b/dev/build/ProjectInstructions/standards/CODEX_WORKFLOW_COMMANDS.md
63-
@@ -25,9 +25,26 @@ Start of Day Golf
64-
65-
Commands using non-canonical team names are invalid because active team names are Owner, Alfa, Bravo, Charlie, Delta, and Golf.
66-
67-
-Start of Day is read-only discovery only.
68-
+Start of Day is read-only discovery plus baseline sync only.
69-
70-
-Start of Day must not require the current branch to be `main`.
71-
+Start of Day requires:
72-
+- current branch is `main`
73-
+- worktree is clean
74-
+- `git fetch origin`
75-
+- `git pull --ff-only origin main`
76-
+- local `main` and `origin/main` are synced
77-
+
78-
+Allowed Start of Day git sync actions:
79-
+- `git fetch origin`
80-
+- `git pull --ff-only origin main`
81-
+
82-
+If the current branch is not `main`, Start of Day must HARD STOP and return these correction steps:
83-
+
84-
+```text
85-
+git switch main
86-
+git pull --ff-only origin main
87-
+Start of Day {Team}
88-
+```
89-
90-
Start of Day must not:
91-
- create a branch
1+
diff --git a/.gitignore b/.gitignore
2+
index 3ac762a04..1207a12fc 100644
3+
--- a/.gitignore
4+
+++ b/.gitignore
5+
@@ -48,8 +48,8 @@ docs/dev/reports/playwright_v8_coverage_report.txt
6+
*.mp4
7+
*.mkv
8+
9+
-.env
10+
-.env.*
11+
+# Environment files
12+
+.env*
13+
!.env.example
14+
!.env.sample
15+
!.env.template

0 commit comments

Comments
 (0)