Skip to content

Commit 8b39dc2

Browse files
authored
conformance: MQ L4/5 range, syntax token separators, typed-om position values, and feasibility consensus (#3)
1 parent 76fbe38 commit 8b39dc2

77 files changed

Lines changed: 4314 additions & 2130 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.agents/skills/champ/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ The subagent persona is a superstar senior SWE. They love concrete specification
1616

1717
## Subagent Workflow
1818
1. **Pick a Task**: Select an uncompleted task from `PLAN.md`. Mark it as in progress `[/]` using the `plan-manager` skill if available.
19-
2. **Diagnostic Cluster Triage (for WPT tasks)**: When tackling WPT conformance tasks, run `node scripts/wpt_cluster_failures.ts --spec=<target>` to inspect top failure clusters and prioritize high-frequency patterns.
19+
2. **Diagnostic Cluster Triage (for WPT tasks)**: When tackling WPT conformance tasks, run `node scripts/wpt/node/cluster.ts --spec=<target>` to inspect top failure clusters and prioritize high-frequency patterns.
2020
3. **Red Phase**: Write a failing test in the appropriate test file (or create a new one in `tests/`) that demonstrates the missing feature or bug.
2121
4. **Green Phase**: Implement the code in `src/` to make the test pass, adding explicit spec anchor citations in comments.
2222
5. **Verify**: Run the targeted test to ensure it passes.

.agents/skills/coherence-auditor/SKILL.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,11 @@ Use this skill when you need to audit the consistency, link integrity, codebase
4343
- **Generated Code Separation**: Verify that all machine-generated spec files reside in `src/data/gen/` (e.g. `src/data/gen/properties.ts`, `src/data/gen/units.ts`, etc.). Flag any generated files placed directly in `src/data/` or other non-gen folders.
4444
- **Extraction Scripts Directory**: Verify that all external test suite extraction scripts reside in `scripts/external_suites/` and use the `extract_<suite>.ts` naming pattern. Flag any extraction scripts placed directly in `scripts/` or missing the `extract_` verb prefix.
4545
- **Filename Case Standard**:
46-
- Enforce **snake_case** for administrative/utility scripts in `scripts/` (e.g. `prune_resolved_failures.ts`, `generate_all.ts`) and codegen scripts in `scripts/codegen/`.
46+
- Enforce **snake_case** for administrative/utility scripts in `scripts/` (e.g. `scripts/baselines/prune_resolved_failures.ts`, `scripts/codegen/generate_all.ts`) and codegen scripts in `scripts/codegen/`.
4747
- Enforce **kebab-case** for public package files, test runners in `tests/` (e.g., `external-lightning.test.ts`, `wpt-extracted.test.ts`), JSON fixtures in `tests/fixtures/`, and baseline files (which must reside in `tests/fixtures/baselines/`).
4848
- Flag any deviations or inconsistent mixing of case conventions within the same category.
49-
- **Test Failure & Skip Taxonomy**: Audit terms used for skipping/categorizing failures (`exclude` for unexecutable HTML suites, `knownFailures` for baseline failures checked and pruned by `prune_resolved_failures.ts`, and `knownSkips` for mapped skips with explicit spec reason strings).
50-
- **Script & Command Alignment**: Ensure script filenames correspond logically to npm script names in `package.json` (e.g. `codegen` $\to$ `scripts/generate_all.ts`, `external:extract` $\to$ `scripts/extract_external_suites.ts`, `baselines:prune` $\to$ `scripts/prune_resolved_failures.ts`).
49+
- **Test Failure & Skip Taxonomy**: Audit terms used for skipping/categorizing failures (`exclude` for unexecutable HTML suites, `knownFailures` for baseline failures checked and pruned by `scripts/baselines/prune_resolved_failures.ts`, and `knownSkips` for mapped skips with explicit spec reason strings).
50+
- **Script & Command Alignment**: Ensure script filenames correspond logically to npm script names in `package.json` (e.g. `codegen` $\to$ `scripts/codegen/generate_all.ts`, `external:extract` $\to$ `scripts/external_suites/extract_all.ts`, `baselines:prune` $\to$ `scripts/baselines/prune_resolved_failures.ts`).
5151

5252
3. **Index & Link Integrity Verification**:
5353
- **Automated Link & Spec Path Validation**: Run `node .agents/skills/coherence-auditor/scripts/validate_links.ts` to automatically detect broken relative markdown links, invalid spec submodule paths, and missing script/source file references across all docs.

.gemini/context.md

Lines changed: 0 additions & 49 deletions
This file was deleted.

.vscode/settings.json

Lines changed: 0 additions & 6 deletions
This file was deleted.

AGENTS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ We intentionally deviate from some specifications for pragmatism, performance, o
4646
### Automation Over Hardcoding
4747
As CSS specifications evolve, this codebase must evolve with them. We prioritize automation over manual maintenance for spec-derived data.
4848
- **Rule**: Whenever adding support for new properties, values, units, or features that are documented in external data sources (like `mdn-data` or `@webref/css`), ALWAYS prefer updating or creating a code generation script in `scripts/codegen/` rather than hardcoding lists in implementation files.
49-
- **Rule**: Ensure that any new generation script is added to the master script `scripts/generate_all.ts` and that the `pnpm run codegen` command functions correctly.
49+
- **Rule**: Ensure that any new generation script is added to the master script `scripts/codegen/generate_all.ts` and that the `pnpm run codegen` command functions correctly.
5050
- **Rule**: The `maintain` script in `package.json` must always include `pnpm run codegen` to ensure that updating submodules automatically updates our generated data.
5151

5252
## Instructions for Agents

LOOP.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ graph TD
2727
* *Role*: Implements features, writes tests, runs `pnpm run preflight`, and commits changes to git following the [champ skill](file:///usr/local/google/home/paulirish/code/cssom/.agents/skills/champ/SKILL.md).
2828
* *Standards*:
2929
* **Spec Anchor Citations**: Must cite exact normative specification anchors in code comments (e.g. `// cssom-1 § 6.5.3 #insert-a-css-rule`) mapping implementation to standard algorithms.
30-
* **Mandatory Pre/Post Cluster Triage & Delta Reconciliation**: When working on WPT conformance waves, start with `node scripts/wpt_cluster_failures.ts --spec=<target>` to record the baseline failure cluster. After implementation, run `wpt_cluster_failures` again. If the test increase is noticeably smaller than the targeted cluster, the developer **MUST diagnose the top remaining failure cluster** and fix any near-miss harness/serialization gaps (e.g. computed color formatting) before declaring the phase complete.
30+
* **Mandatory Pre/Post Cluster Triage & Delta Reconciliation**: When working on WPT conformance waves, start with `node scripts/wpt/node/cluster.ts --spec=<target>` to record the baseline failure cluster. After implementation, run `cluster.ts` again. If the test increase is noticeably smaller than the targeted cluster, the developer **MUST diagnose the top remaining failure cluster** and fix any near-miss harness/serialization gaps (e.g. computed color formatting) before declaring the phase complete.
3131
* *Constraint*: Naturally optimistic. Wants compilation and test runs to pass as quickly as possible.
3232
3. **The Reviewer (`codex_reviewer_cmd`)**:
3333
* *Role*: Senior engineer persona. Has command execution permissions and runs `git show HEAD` to audit styling, typing, spec citations, and safety.

MAINTENANCE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ This runs `git submodule update --init --remote && pnpm run submodules:update` t
3636
```bash
3737
pnpm run fixtures:generate
3838
```
39-
This runs `node scripts/extract_external_suites.ts`.
39+
This runs `node scripts/external_suites/extract_all.ts`.
4040

4141
**3. Run Tests:**
4242
```bash

0 commit comments

Comments
 (0)