Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
40 changes: 20 additions & 20 deletions docs_build/dev/BUILD_PR.md
Original file line number Diff line number Diff line change
@@ -1,43 +1,43 @@
# PR_26177_004-shared-color-foundation
# PR_26177_005-shared-text-foundation

## Purpose

Add a small shared color foundation.
Add a small shared text foundation.

## Source Of Truth

This `BUILD_PR.md`, `PLAN_PR.md`, and the user request are the source of truth for `PR_26177_004-shared-color-foundation`.
This `BUILD_PR.md`, `PLAN_PR.md`, and the user request are the source of truth for `PR_26177_005-shared-text-foundation`.

## Stack

- Base branch: `PR_26177_003-shared-geometry-foundation`
- Base branch: `PR_26177_004-shared-color-foundation`

## Exact Scope

- Add `src/shared/color/` foundation.
- Include hex/rgb/hsl conversion, clamp, lerp/blend helpers, luminance/contrast basics.
- Add targeted tests for the shared color area.
- No page styling changes.
- Add `src/shared/text/` foundation.
- Include safe string helpers such as slugify, casing, truncate, escapeHtml, and normalizeWhitespace.
- Add targeted tests for the shared text area.
- No copy rewrites outside tests/docs.
- Create required Codex reports under `docs_build/dev/reports/`.
- Create repo-structured delta ZIP under `tmp/`.

## Exact Targets

- `docs_build/dev/PLAN_PR.md`
- `docs_build/dev/BUILD_PR.md`
- `src/shared/color/color.js`
- `tests/shared/ColorFoundation.test.mjs`
- `docs_build/dev/reports/PR_26177_004-shared-color-foundation.md`
- `docs_build/dev/reports/PR_26177_004-shared-color-foundation_branch-validation.md`
- `docs_build/dev/reports/PR_26177_004-shared-color-foundation_requirement-checklist.md`
- `docs_build/dev/reports/PR_26177_004-shared-color-foundation_validation-lane.md`
- `docs_build/dev/reports/PR_26177_004-shared-color-foundation_manual-validation-notes.md`
- `src/shared/text/text.js`
- `tests/shared/TextFoundation.test.mjs`
- `docs_build/dev/reports/PR_26177_005-shared-text-foundation.md`
- `docs_build/dev/reports/PR_26177_005-shared-text-foundation_branch-validation.md`
- `docs_build/dev/reports/PR_26177_005-shared-text-foundation_requirement-checklist.md`
- `docs_build/dev/reports/PR_26177_005-shared-text-foundation_validation-lane.md`
- `docs_build/dev/reports/PR_26177_005-shared-text-foundation_manual-validation-notes.md`
- `docs_build/dev/reports/codex_review.diff`
- `docs_build/dev/reports/codex_changed_files.txt`

## Out Of Scope

- No page styling changes.
- No copy rewrites outside tests/docs.
- No browser-owned product data.
- No runtime UI changes.
- No browser storage changes.
Expand All @@ -51,9 +51,9 @@ This `BUILD_PR.md`, `PLAN_PR.md`, and the user request are the source of truth f
Run exactly:

```powershell
node ./scripts/run-node-test-files.mjs tests/shared/ColorFoundation.test.mjs
node --check src/shared/color/color.js
node --check tests/shared/ColorFoundation.test.mjs
node ./scripts/run-node-test-files.mjs tests/shared/TextFoundation.test.mjs
node --check src/shared/text/text.js
node --check tests/shared/TextFoundation.test.mjs
git diff --check
```

Expand All @@ -62,5 +62,5 @@ git diff --check
Create repo-structured delta ZIP:

```text
tmp/PR_26177_004-shared-color-foundation_delta.zip
tmp/PR_26177_005-shared-text-foundation_delta.zip
```
17 changes: 8 additions & 9 deletions docs_build/dev/PLAN_PR.md
Original file line number Diff line number Diff line change
@@ -1,23 +1,22 @@
# PLAN_PR: PR_26177_004-shared-color-foundation
# PLAN_PR: PR_26177_005-shared-text-foundation

## Purpose

Add a small shared color foundation.
Add a small shared text foundation.

## Scope

- Add `src/shared/color/` foundation.
- Include hex/rgb/hsl conversion helpers.
- Include clamp, lerp/blend helpers, luminance, and contrast basics.
- Add `src/shared/text/` foundation.
- Include safe string helpers for whitespace normalization, slug/casing, truncation, and HTML escaping.
- Add targeted tests.
- No page styling changes.
- No copy rewrites outside tests/docs.
- No browser-owned product data.
- No runtime UI changes.
- No unrelated cleanup.

## Implementation Plan

1. Add `src/shared/color/color.js`.
2. Add `tests/shared/ColorFoundation.test.mjs`.
3. Validate conversion, blending, luminance, and contrast helpers.
1. Add `src/shared/text/text.js`.
2. Add `tests/shared/TextFoundation.test.mjs`.
3. Validate safe string helpers.
4. Produce required Codex reports and repo-structured ZIP.
23 changes: 23 additions & 0 deletions docs_build/dev/reports/PR_26177_005-shared-text-foundation.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# PR_26177_005-shared-text-foundation

Date: 2026-06-26
Scope: Shared text foundation
Status: PASS

## Summary

- Added `src/shared/text/text.js`.
- Added safe string helpers for whitespace normalization, slug/casing, truncation, and HTML escaping.
- Added targeted tests in `tests/shared/TextFoundation.test.mjs`.
- No copy rewrites, runtime UI, browser product-data ownership, API/database, or unrelated cleanup changes were made.

## Validation

- PASS: `node ./scripts/run-node-test-files.mjs tests/shared/TextFoundation.test.mjs`.
- PASS: `node --check src/shared/text/text.js`.
- PASS: `node --check tests/shared/TextFoundation.test.mjs`.
- PASS: `git diff --check`.

## Artifact

- `tmp/PR_26177_005-shared-text-foundation_delta.zip`
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
# PR_26177_005-shared-text-foundation Branch Validation

- PASS: Current branch is `PR_26177_005-shared-text-foundation`.
- PASS: Branch is stacked on `PR_26177_004-shared-color-foundation`.
- PASS: Scope is documentation/report files plus shared text code and tests only.
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
# PR_26177_005-shared-text-foundation Manual Validation Notes

- Reviewed scope against the user request and BUILD document.
- Confirmed the change does not alter UI, storage, API, database, scheduler, or engine behavior.
- Confirmed no `start_of_day` files were modified.
- Confirmed ZIP artifact path: `tmp/PR_26177_005-shared-text-foundation_delta.zip`.
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# PR_26177_005-shared-text-foundation Requirement Checklist

- PASS: One PR purpose only.
- PASS: Smallest valid scoped change.
- PASS: No unrelated cleanup.
- PASS: Did not modify `start_of_day` folders.
- PASS: Did not add browser product-data ownership.
- PASS: Used ES modules consistent with repo style.
- PASS: Added `src/shared/text/` foundation.
- PASS: Included `slugify`, casing, `truncate`, `escapeHtml`, and `normalizeWhitespace` helpers.
- PASS: No copy rewrites outside tests/docs.
- PASS: Added targeted tests for the shared text area.
- PASS: Did not run full samples smoke by default.
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
# PR_26177_005-shared-text-foundation Validation Lane

- PASS: `node ./scripts/run-node-test-files.mjs tests/shared/TextFoundation.test.mjs`.
- PASS: `node --check src/shared/text/text.js`.
- PASS: `node --check tests/shared/TextFoundation.test.mjs`.
- PASS: `git diff --check`.

Full samples smoke was intentionally not run because this PR changes only a shared helper module and targeted tests.
14 changes: 7 additions & 7 deletions docs_build/dev/reports/codex_changed_files.txt
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
docs_build/dev/BUILD_PR.md
docs_build/dev/PLAN_PR.md
docs_build/dev/reports/PR_26177_004-shared-color-foundation.md
docs_build/dev/reports/PR_26177_004-shared-color-foundation_branch-validation.md
docs_build/dev/reports/PR_26177_004-shared-color-foundation_manual-validation-notes.md
docs_build/dev/reports/PR_26177_004-shared-color-foundation_requirement-checklist.md
docs_build/dev/reports/PR_26177_004-shared-color-foundation_validation-lane.md
docs_build/dev/reports/PR_26177_005-shared-text-foundation.md
docs_build/dev/reports/PR_26177_005-shared-text-foundation_branch-validation.md
docs_build/dev/reports/PR_26177_005-shared-text-foundation_manual-validation-notes.md
docs_build/dev/reports/PR_26177_005-shared-text-foundation_requirement-checklist.md
docs_build/dev/reports/PR_26177_005-shared-text-foundation_validation-lane.md
docs_build/dev/reports/codex_changed_files.txt
docs_build/dev/reports/codex_review.diff
src/shared/color/color.js
tests/shared/ColorFoundation.test.mjs
src/shared/text/text.js
tests/shared/TextFoundation.test.mjs
Loading
Loading