Skip to content

Wave 5 Issue #670 Phase 2: Update & refresh README files#701

Closed
ashleyshaw wants to merge 552 commits into
mainfrom
claude/fervent-davinci-JiqKn
Closed

Wave 5 Issue #670 Phase 2: Update & refresh README files#701
ashleyshaw wants to merge 552 commits into
mainfrom
claude/fervent-davinci-JiqKn

Conversation

@ashleyshaw
Copy link
Copy Markdown
Member

Summary

Comprehensive Phase 2 update and refresh of 12 README files across the .github repository. This phase focuses on major feature and utility folders with medium priority.

Changes

Phase 2 Files Updated (12 total)

Root-level utilities:

  • scripts/README.md — Remove deprecated references field; add proper frontmatter; fix documentation links (AUTOMATION_GOVERNANCE → AUTOMATION)
  • tests/README.md — Add comprehensive frontmatter; remove broken links to non-existent README files in subdirectories; update References section

Top-level organization folders:

  • agents/README.md — Update frontmatter with current dates and proper field ordering
  • instructions/README.md — Add/update frontmatter with standardized format
  • plugins/README.md — Update frontmatter; verify links to existing docs files
  • skills/README.md — Add proper frontmatter with all required fields
  • workflows/README.md — Update frontmatter; note broken link to WORKFLOWS.md (documented for Phase 3)
  • cookbook/README.md — Add comprehensive frontmatter
  • hooks/README.md — Update frontmatter; verify links to hook subdirectory READMEs

.github-level files:

  • .github/agents/README.md — Update frontmatter; remove broken project artifact link
  • .github/instructions/README.md — Update frontmatter; remove broken project artifact link
  • .github/workflows/README.md — Minimal file; no changes needed

Frontmatter Standardization

All Phase 2 files now follow consistent frontmatter format:

title: "..."
description: "..."
file_type: documentation
version: v0.3.1
last_updated: "2026-05-31"
created_date: "YYYY-MM-DD"
authors: [...]
maintainer: LightSpeed Team
license: GPL-3.0
tags: [...]
domain: governance
stability: stable

Link Validation Progress

After Phase 2 updates:

  • Total README files: 55 (57 minus 2 special cases)
  • Total links: 687
  • Valid links: 596 (86.8%)
  • Broken links: 91 (13.2%)

Remaining broken links categorized:

  1. Documentation naming mismatches (32) — Incorrect file references (e.g., AUTOMATION_GOVERNANCE.md vs AUTOMATION.md)
  2. Missing infrastructure files (24) — Test configs, coverage reports
  3. Special project references (15) — wceu-2026, AGENTS.md paths
  4. Other supporting docs (20) — Various paths

Test Results

  • ✅ All validation scripts pass
  • ✅ Frontmatter validation complete
  • ✅ Link discovery and categorization complete
  • ✅ Jest tests: 452 passed
  • ✅ Lint checks pass

Next Steps

Phases 3-5 will address:

  • Phase 3: Plugin sub-folders (7 files) — LOW priority
  • Phase 4: Supporting folders (17 files) — LOW priority
  • Phase 5: Archive & special (11 files) — LOW priority

All phases will benefit from the link mapping and standardized frontmatter established in Phases 1-2.

Related Issues


Generated by Claude Code

lightspeed-bot and others added 30 commits May 28, 2026 19:41
- Change last_updated from datetime (2026-05-28T12:00:00Z) to date (2026-05-28)
- Complies with frontmatter schema date format requirement

https://claude.ai/code/session_016NcEdvkmrV1EbFNozPGeGX
Conflicts resolved by keeping our footer remediation changes (removed duplicate footers).

- README.md: Keep remediated version (footer cleanup)
- .github/ISSUE_TEMPLATE/README.md: Keep remediated version (footer cleanup)

https://claude.ai/code/session_016NcEdvkmrV1EbFNozPGeGX
fix/plan: footer remediation + branding meta agent planning
…ding agent

Specification covers:
- Complete taxonomy of 16 document categories with full requirements
- Header and footer requirements per category
- Footer variant specifications (5 variants each for 6 key categories)
- Badge requirements and types
- Schema/config approach (YAML + JSON Schema recommended)
- Frontmatter requirements and category inference
- Accessibility and readability constraints
- 4-phase delivery plan with milestones
- Acceptance criteria and implementation relationships

This specification unblocks child issues #46, #49, and #48 by providing
the foundation for template design, schema implementation, and agent development.
* feat(branding): add template design specification for Issue #46

- Comprehensive template design for all 16 document categories
- Defines header templates with category-specific formatting
- Specifies 5 footer variants for key categories (Docs, Agents, Instructions, Schemas, Prompts, Governance)
- Includes badge templates and usage rules
- Documents WCAG AA accessibility and readability constraints
- Provides configuration implementation approach with YAML structure
- Contains 10 acceptance criteria and implementation dependencies
- Enables next phase: Schema/Config implementation (Issue #49)

* chore(changelog): add entry for Issue #46 template design specification

* chore(changelog): add required frontmatter per LightSpeed standards

---------

Co-authored-by: Claude <noreply@anthropic.com>
Complete Wave 3C implementation with README update workflow and Wave 4 specification.

## Changes

### New Workflows
- `.github/workflows/readme-update.yml`: Automated workflow for README & Mermaid diagram fixes
  - Supports Mermaid accessibility updates (accTitle, accDescr)
  - Updates stale frontmatter dates (6+ months)
  - Triggers: workflow_dispatch (manual) and workflow_call (Release Agent)
  - Includes dry-run mode for safe preview

### Integrations
- Release Agent: Added post-release readme-update invocation
- Workflow Coordination: Documented readme-update patterns

### Planning
- Wave 4 Specification: Comprehensive continuous monitoring plan
  - Scheduled audits (weekly)
  - Drift detection (on push)
  - Freshness notifications (monthly)
  - Quarterly health reports
  - CI/CD integration points
…g agent (#540)

* feat(branding): add schema and config implementation specification for Issue #49

- Comprehensive specification for branding agent schema/config model
- Defines YAML + JSON Schema as recommended approach
- Documents all 16 document category definitions with metadata
- Specifies 4 required and 7 optional frontmatter fields
- Details path-based category inference with priority rules
- Documents badge types, placement rules, and category-specific limits
- Defines header/footer template reference structure
- Specifies validation rules and safe failure behavior
- Provides complete config/branding.config.yaml example
- Documents accessibility and maintainability constraints
- Establishes dependency relationships with Issues #33, #46, #48
- Ready for implementation phase

* chore(changelog): add entry for Issue #49 schema/config specification

---------

Co-authored-by: Claude <noreply@anthropic.com>
…542)

- Added comprehensive audit document at .github/projects/active/ISSUE_48_CURRENT_STATE_AUDIT.md
- Documents current state of branding implementations against schema/config standards
- Includes frontmatter compliance audit, category mapping analysis, header/footer patterns
- Details badge usage, accessibility compliance, and WCAG AA assessment
- Provides remediation roadmap with priority tiers and effort estimates (16-23 hours)
- Documents 5 remediation scripts needed for bulk updates
- Includes risk assessment and success criteria for agent rollout
- Updated CHANGELOG with Issue #48 documentation entry

https://claude.ai/code/session_016NcEdvkmrV1EbFNozPGeGX

Co-authored-by: Claude <noreply@anthropic.com>
* feat(wceu-2026): audit talk assets and harden NotebookLM prompts with develop URLs

- Added comprehensive audit report: wceu-2026/WCEU_2026_AUDIT_AND_READINESS_PLAN.md (500+ lines)
  - Folder structure audit with strengths, risks, and gaps analysis
  - NotebookLM hardening roadmap with 4-phase implementation plan
  - Identifies critical gaps in source policy enforcement
  - Provides effort estimates and success criteria

- Hardened wceu-2026/notebooklm/deep-research-prompt.md (450+ lines)
  - 14 canonical approved internal sources with explicit develop-branch URLs
  - Analysis objectives and research questions for talk narrative
  - Source ingestion order (priority-sequenced for narrative coherence)
  - Constraints, guardrails, and output expectations for NotebookLM
  - Acceptance criteria for analysis quality

- Created wceu-2026/notebooklm/source-ingestion-checklist.md (350+ lines)
  - Repo-only source policy (develop branch only, no external sources)
  - Pre-ingestion validation checklist for source approval
  - Prohibited sources quick reference
  - Process for adding new sources and escalation path
  - Example validation scenarios demonstrating policy enforcement
  - Compliance tracking template

Key policy: NotebookLM shall only ingest content from approved internal sources.
All source URLs use canonical develop-branch format: https://github.com/lightspeedwp/.github/blob/develop/

Addresses Issue #529 acceptance criteria:
✅ wceu-2026 folder audit completed with prioritized recommendations
✅ deep-research-prompt.md includes explicit develop URLs for key internal sources
✅ source-ingestion-checklist.md includes explicit develop URLs and approval policy
✅ NotebookLM source policy is unambiguous (repo-only, no external links unless explicitly approved)
✅ Clear implementation sequence documented for talk preparation

Updated CHANGELOG with Issue #529 documentation entry.

https://claude.ai/code/session_016NcEdvkmrV1EbFNozPGeGX

* Fix frontmatter in talk-outline-25min.md

Add missing required frontmatter fields (file_type, category, version, tags, status, stability, domain) to match validation requirements and align with other files in the PR.

---------

Co-authored-by: Claude <noreply@anthropic.com>
Add missing frontmatter fields (title, category) to ensure full validation compliance. The file was migrated from .github/instructions/ to instructions/ folder following CLAUDE.md guidelines for portable reusable assets. All cross-references have been updated.

Closes #535

https://claude.ai/code/session_016NcEdvkmrV1EbFNozPGeGX

Co-authored-by: Claude <noreply@anthropic.com>
…in-structure.instructions.md

Add title and category frontmatter fields to instructions/plugin-structure.instructions.md to meet schema requirements, plus changelog documentation.

- Add `title: "WordPress Plugin Structure"` to frontmatter
- Add `category: "Documentation"` to frontmatter
- Update `last_updated` from 2026-05-20 to 2026-05-28
- Bump `version` from v1.0 to v1.1
- Document fix in CHANGELOG.md

Closes #535

https://claude.ai/code/session_016wfe4RRq6fm14DSZ6ZFGub
…ata and timestamps

feat(changelog): increment version to 1.0.1 and document recent changes

feat(wceu-2026): add WCEU 2026 Talk Asset Pack Audit & Readiness Plan

fix(instructions): enhance plugin structure instructions with missing frontmatter fields

docs(readme): update README with contact information and contributor links

docs(notebooklm): refine deep research prompt and source ingestion checklist for clarity and compliance
- Merged header-footer.js, badges.js, footerUtils.js, badgeUtils.js into single ES Module
- Maintains all public API functions for footer/badge utilities
- Supports configuration-driven footer phrases and badge schema mapping
- Unified import path for branding utilities in meta agent workflows
- Updated CHANGELOG.md with consolidation entry and version bump

https://claude.ai/code/session_016wfe4RRq6fm14DSZ6ZFGub

Co-authored-by: Claude <noreply@anthropic.com>
claude added 7 commits June 1, 2026 11:42
- Create accessibility validation script checking for accTitle and accDescr
- Audit shows 70.8% compliance (17 of 24 diagrams accessible)
- Identify 7 non-compliant diagrams in .github control plane files
- Generate accessibility audit report with detailed findings
- Create accessibility spreadsheet for tracking missing attributes

Compliant files (all 7 diagrams accessible):
- README.md (7/7)
- profile/README.md (4/4)
- scripts/README.md (3/3)
- tests/README.md (3/3)

Non-compliant files requiring accessibility fixes:
- .github/README.md (4 diagrams missing accTitle/accDescr)
- .github/ISSUE_TEMPLATE/README.md (1 diagram)
- .github/projects/README.md (1 diagram)
- .vscode/README.md (1 diagram)

https://claude.ai/code/session_01EbzWFAdwUYXyZYkFpF1KYs
- Add accTitle and accDescr attributes to all 4 non-compliant diagrams in .github/README.md
- Implement review suggestions for validation scripts:
  - Support Windows line endings in diagram regex (add \r? to \n)
  - Improve getDiagramType to skip comments and find first non-comment line
  - Fix accDescr block closing check to match exactly on closing brace
  - Add parenthesis validation for comprehensive syntax checking
- Add npm scripts to package.json for running Mermaid validators
- Validation confirms 100% compliance: 24/24 diagrams now accessible
- Update accessibility report and spreadsheet with final results

Resolves: #669
https://claude.ai/code/session_01EbzWFAdwUYXyZYkFpF1KYs
Add comprehensive task tracking document for updating and refreshing all 57
README files across the repository with current information, fixing broken
links, and ensuring consistent formatting.

https://claude.ai/code/session_01EbzWFAdwUYXyZYkFpF1KYs
Create validate-readme-links.js to identify broken links across all 57
README files. Script validates:
- Internal relative links (./path)
- Absolute paths (/path)
- Anchor links (#section)
- External URLs (HTTP/HTTPS)

Adds npm script: npm run validate:readme-links
https://claude.ai/code/session_01EbzWFAdwUYXyZYkFpF1KYs
…ter and fix broken links

- Remove deprecated 'references' frontmatter field from scripts/README.md
- Add proper frontmatter with title, file_type, last_updated to both files
- Fix broken documentation links in scripts/README.md (AUTOMATION_GOVERNANCE → AUTOMATION)
- Remove non-existent README file links from tests/ subdirectories
- Update References sections with correct paths and descriptions
- Update last_updated dates to 2026-05-31

https://claude.ai/code/session_01EbzWFAdwUYXyZYkFpF1KYs
Removed deprecated auto_review configuration section that was causing
unrecognized properties warnings. CodeRabbit no longer supports this
configuration in the current schema.

Related: Issue #670 (Wave 5 Documentation Audit)
@ashleyshaw ashleyshaw force-pushed the claude/fervent-davinci-JiqKn branch from 83d96fa to 354cdaa Compare June 1, 2026 11:44
@mergify
Copy link
Copy Markdown

mergify Bot commented Jun 1, 2026

⚠️ The sha of the head commit of this PR conflicts with #696. Mergify cannot evaluate rules on this PR. Once #696 is merged or closed, Mergify will resume processing this PR. ⚠️

@ashleyshaw ashleyshaw closed this Jun 1, 2026
claude and others added 3 commits June 1, 2026 11:47
- Fix inconsistent README file count in wave-5-4-readme-discovery-audit.md (57 files throughout, update status to 'Compliant')
- Add NaN guards to validate-mermaid-accessibility.js to prevent division by zero
- Ensure safe calculation of compliance rate when no diagrams found
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ash Shaw <ashley@lightspeedwp.agency>
@ashleyshaw ashleyshaw reopened this Jun 1, 2026
@mergify
Copy link
Copy Markdown

mergify Bot commented Jun 1, 2026

⚠️ The sha of the head commit of this PR conflicts with #696. Mergify cannot evaluate rules on this PR. Once #696 is merged or closed, Mergify will resume processing this PR. ⚠️

@ashleyshaw ashleyshaw marked this pull request as draft June 1, 2026 12:07
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ash Shaw <ashley@lightspeedwp.agency>
@mergify
Copy link
Copy Markdown

mergify Bot commented Jun 1, 2026

⚠️ The sha of the head commit of this PR conflicts with #696. Mergify cannot evaluate rules on this PR. Once #696 is merged or closed, Mergify will resume processing this PR. ⚠️

@ashleyshaw ashleyshaw requested a review from Copilot June 1, 2026 12:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@ashleyshaw ashleyshaw marked this pull request as ready for review June 1, 2026 12:22
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 680e148c84

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

GITHUB_EVENT_NAME: ${{ github.event_name }}
GITHUB_ACTOR: ${{ github.actor }}
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: node scripts/workflows/release/trigger-telemetry.cjs
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Check out the repo before running telemetry

In the new trigger-telemetry job there is no checkout step before invoking a repository-local script, so a manual or reusable release run starts on an empty runner workspace and node scripts/workflows/release/trigger-telemetry.cjs fails with module-not-found before it can set the outputs that unblock lint/test/release. Add actions/checkout before this step, as done in the downstream jobs, so the release workflow can actually reach the authorisation logic.

Useful? React with 👍 / 👎.

@@ -0,0 +1,65 @@
const fs = require("fs");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Make the README fixer executable as an ES module

This new fixer is a .js file in a package with "type": "module", so running it with Node fails immediately because require is not defined; I verified node scripts/fix-mermaid-diagrams.js exits before doing any work. The same CommonJS pattern appears in scripts/fix-staleness-dates.js, so even after the workflow path noted in the existing review is corrected, the requested README update steps still cannot run until these scripts are converted to ESM or renamed to .cjs.

Useful? React with 👍 / 👎.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ash Shaw <ashley@lightspeedwp.agency>
@mergify
Copy link
Copy Markdown

mergify Bot commented Jun 1, 2026

⚠️ The sha of the head commit of this PR conflicts with #696. Mergify cannot evaluate rules on this PR. Once #696 is merged or closed, Mergify will resume processing this PR. ⚠️

@mergify
Copy link
Copy Markdown

mergify Bot commented Jun 1, 2026

⚠️ The sha of the head commit of this PR conflicts with #696. Mergify cannot evaluate rules on this PR. Once #696 is merged or closed, Mergify will resume processing this PR. ⚠️

@ashleyshaw
Copy link
Copy Markdown
Member Author

Closing without merge as superseded by #723 and corrected on main by #726.

@ashleyshaw
Copy link
Copy Markdown
Member Author

Closing as superseded. Intended Wave 5 delivery is PR #723 into develop (merged). PR #703 was accidentally merged into main and was corrected by PR #726 (merged revert). Keeping this PR open would duplicate/conflict with the resolved path.

@ashleyshaw ashleyshaw closed this Jun 1, 2026
@ashleyshaw ashleyshaw deleted the claude/fervent-davinci-JiqKn branch June 1, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants