PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ToolboxAid merged 1 commit into from
Jun 22, 2026
Merged

PR_26172_CHARLIE_002-test-results-artifact-cleanup #94

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down
129 changes: 129 additions & 0 deletions docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.
3 changes: 2 additions & 1 deletion docs_build/dev/reports/codex_changed_files.txt
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
PR_26172_CHARLIE_002-test-results-artifact-cleanup#94

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

 Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and
privacy statement. We’ll occasionally send you account related emails.

Already on GitHub?
Sign in
to your account

 Merged

ToolboxAid
merged 1 commit into

from

Jun 22, 2026
 Merged


PR_26172_CHARLIE_002-test-results-artifact-cleanup
#94
Show file tree
Hide file tree
 Changes from all commits
Commits

Show all changes


1 commit


Select commit


File filter

Filter by extension
Filter by extension
Conversations

Failed to load comments. 
Loading
Jump to

Jump to file


Failed to load files. 
Loading
 Diff view

 Diff view


4 changes: 3 additions & 1 deletion
.gitignore

View file


Open in desktop

Original file line numberDiff line numberDiff line change
Expand Up@@ -13,10 +13,12 @@
# Ignore the NEXT_COMMAND.txt file
NEXT_COMMAND.txt

# Ignore the tests results and temporary files
# Ignore generated dependency, test result, and temporary files
node_modules/
tests/results/
tests/results/**
tmp/test-results/
tmp/test-results/**
tmp/
scripts/untracked/
projects/
Expand Down

129 changes: 129 additions & 0 deletions
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,129 @@
# PR_26172_CHARLIE_002 Test Results Artifact Cleanup

## Scope

Clean up generated test result artifacts under `tests/results/` after the Charlie repository compliance audit identified that path as a high-priority cleanup candidate.

Source audit:

- `docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md`

This PR does not modify runtime source and does not move unrelated tests.

## Team Ownership

- TEAM token: CHARLIE
- Ownership classification: governance / repository hygiene / diagnostics
- TEAM ownership result: PASS

## Branch Validation

| Requirement | Status | Evidence |
| --- | --- | --- |
| Started from latest main | PASS | `main` was pulled before branch creation; source commit `f2b50ac9d79256df3a7716ac4eff21f3a4303bb3`. |
| Worktree clean before branch | PASS | `git status --short` returned no output before branch creation. |
| Local/origin sync before branch | PASS | `git rev-list --left-right --count HEAD...origin/main` returned `0 0`. |
| PR branch created from main | PASS | Branch `pr/26172-CHARLIE-002-test-results-artifact-cleanup` was created from latest `main`. |

## Files Reviewed

`git ls-files tests/results` returned no tracked files.

The local ignored `tests/results/` folder contained generated Playwright/report output:

- `tests/results/artifacts/.last-run.json`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-3c5a9-multi-song-manifest-payload-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-752e4-on-and-timeline-scroll-sync-playwright/trace.zip`
- `tests/results/artifacts/tools-MidiStudioV2-MIDI-St-c50c5-m-Tool-Mode-standalone-save-playwright/trace.zip`
- `tests/results/playwright-results.json`
- `tests/results/report/data/09daf0cfe8750af5e9e5bb22161367f97296f4fd.zip`
- `tests/results/report/data/a9ba8bc1c6a629055b981a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/b1dc1da730cbd5e9adc334a6f385fa4de3e42a79d.zip`
- `tests/results/report/data/c150573559f5367f4ec5724abb7a55798abcdff9.zip`
- `tests/results/report/index.html`
- `tests/results/report/trace/assets/codeMirrorModule-DS0FLvoc.js`
- `tests/results/report/trace/assets/defaultSettingsView-GTWI-W_B.js`
- `tests/results/report/trace/codeMirrorModule.DYBRYzYX.css`
- `tests/results/report/trace/codicon.DCmgc-ay.ttf`
- `tests/results/report/trace/defaultSettingsView.B4dS75f0.css`
- `tests/results/report/trace/index.C5466mMT.js`
- `tests/results/report/trace/index.CzXZzn5A.css`
- `tests/results/report/trace/index.html`
- `tests/results/report/trace/manifest.webmanifest`
- `tests/results/report/trace/playwright-logo.svg`
- `tests/results/report/trace/snapshot.html`
- `tests/results/report/trace/sw.bundle.js`
- `tests/results/report/trace/uiMode.Btcz36p_.css`
- `tests/results/report/trace/uiMode.Vipi55dB.js`
- `tests/results/report/trace/uiMode.html`
- `tests/results/report/trace/xtermModule.DYP7pi_n.css`

## Files Removed Or Retained

| Category | Status | Notes |
| --- | --- | --- |
| Tracked files under `tests/results/` | None removed | No tracked files existed under `tests/results/`. |
| Local ignored generated artifacts under `tests/results/` | Removed from workspace | Removed only after verifying the resolved target path was inside the repository. |
| Active test source | Retained | No active test source was found under `tests/results/`. |
| Fixture or baseline data | Retained | No committed fixture or baseline dependency was found under `tests/results/`. |

## Reference And Dependency Check

| Check | Status | Evidence |
| --- | --- | --- |
| Active tracked files under `tests/results/` | PASS | `git ls-files tests/results` returned no output. |
| Tracked ignored files under `tests/results/` | PASS | `git ls-files -c -i --exclude-standard tests/results` returned no output. |
| Ignored local generated files under `tests/results/` | PASS | `git ls-files -o -i --exclude-standard tests/results` listed only Playwright/report artifacts. |
| Active config uses `tmp/test-results/` | PASS | `playwright.config.cjs` writes output, artifacts, HTML report, and JSON report under `tmp/test-results/`. |
| Active references to `tests/results/` | PASS | Active config/test/docs search returned no required source or fixture dependency. |
| Historical references retained | PASS | References in `archive/` and historical `docs_build/dev/reports/` were not modified. |

## Ignore Rule Changes

Updated `.gitignore` to make generated test-output protection explicit:

- Kept `tests/results/`.
- Added `tests/results/**`.
- Kept `tmp/test-results/`.
- Added `tmp/test-results/**`.
- Confirmed `tmp/` remains ignored.

Ignore probe:

- `git check-ignore -v tests/results/probe.txt` resolves to `.gitignore`.
- `git check-ignore -v tmp/test-results/probe.txt` resolves to `.gitignore`.

## Requirement Checklist

| Requirement | Status | Evidence |
| --- | --- | --- |
| Confirm Project Instructions were reviewed | PASS | Read `docs_build/dev/ProjectInstructions/README.txt`, `PROJECT_INSTRUCTIONS.md`, branch/workflow governance, team ownership, and artifact/reporting rules. |
| Use PR_26172_CHARLIE_001 findings | PASS | This cleanup is based on the P0 `tests/results/` finding. |
| Review `tests/results/` | PASS | Reviewed tracked, ignored, and local generated contents. |
| Confirm generated artifacts, not active source | PASS | Files were Playwright JSON, HTML report, trace assets, and zipped trace/report data. |
| Search references to `tests/results/` files | PASS | No active source/fixture dependency found; historical references retained. |
| Remove tracked generated artifacts if safe | PASS | No tracked generated artifacts existed to remove. |
| Add/update ignore rules | PASS | `.gitignore` now explicitly includes `tests/results/**` and `tmp/test-results/**`. |
| Do not remove active test source | PASS | No active test source removed. |
| Do not modify runtime source | PASS | No runtime source changed. |
| Do not move unrelated tests | PASS | No test files were moved. |
| Stop gate not triggered | PASS | No `tests/results/` file was required as active source, fixture data, or committed baseline data. |
| Create required reports | PASS | `docs_build/dev/reports/codex_review.diff` and `docs_build/dev/reports/codex_changed_files.txt` exist. |
| Create ZIP artifact | PASS | `tmp/PR_26172_CHARLIE_002-test-results-artifact-cleanup_delta.zip` exists. |

## Validation Lane Report

- `git diff --check`: PASS.
- Cleanup limited to generated artifacts under `tests/results/`: PASS.
- Ignore rule prevents recommit: PASS.
- Runtime source files changed: PASS, no runtime source files changed.
- Required reports exist: PASS.
- ZIP artifact exists: PASS.
- Playwright: SKIP, ignore/report-only cleanup with no active test or runtime source changes.
- Samples: SKIP, no sample files changed.

## Manual Validation Notes

- The local ignored `tests/results/` directory was deleted from the workspace only after path verification showed it was inside the repository root.
- Repository history already contains `docs_build/dev/reports/docs_archive_test_output_cleanup_report.md`, which documents the prior migration of generated test output from `tests/results/` to `tmp/test-results/`.
- This PR preserves historical report/archive references and only hardens the active ignore rule.

3 changes: 2 additions & 1 deletion
docs_build/dev/reports/codex_changed_files.txt

View file


Open in desktop

Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
docs_build/dev/reports/PR_26172_CHARLIE_001-repository-compliance-audit.md
.gitignore
docs_build/dev/reports/PR_26172_CHARLIE_002-test-results-artifact-cleanup.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
Loading
Loading