Skip to content

feat(regrade): derive plans from vocabulary seeds#959

Draft
galligan wants to merge 2 commits into
trl-1130-regrade-file-renames-as-governed-transition-facts-withfrom
trl-1134-regrade-seed-to-derived-plan-synthesis-from-minimal-fromto
Draft

feat(regrade): derive plans from vocabulary seeds#959
galligan wants to merge 2 commits into
trl-1130-regrade-file-renames-as-governed-transition-facts-withfrom
trl-1134-regrade-seed-to-derived-plan-synthesis-from-minimal-fromto

Conversation

@galligan

@galligan galligan commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Context

TRL-1134 closes the gap between a minimal vocabulary transition seed and the complete reviewable Regrade plan needed for safe execution. It builds on the three-tier scope contract in #957 and the governed filename/reference-closure engine in #958.

What changed

  • derive morphology, casing, compound/public naming reviews, filename candidates, namespace census, live-topo preserves, and reference-closure evidence from a minimal from/to seed
  • record authored-versus-derived provenance throughout the plan artifact
  • validate trails.lock preserve evidence and return typed validation failures instead of silently dropping malformed data
  • include derived evidence in plan freshness so filename-only drift invalidates saved plans
  • keep direct trail, CLI, and MCP structured outputs equivalent
  • teach the minimal-seed workflow and add release intent for @ontrails/regrade and @ontrails/trails

How to test

  • bun test apps/trails/src/__tests__/regrade.test.ts apps/trails/src/__tests__/mcp.test.ts apps/trails/src/__tests__/regrade-plan-derivation.test.ts packages/regrade/src/downstream/__tests__/vocabulary.test.ts packages/regrade/src/downstream/__tests__/file-renames.test.ts
  • bun run check
  • bun run wayfinder:dogfood
  • bun run publish:check
  • Graphite pre-push suite

Review evidence

  • focused suite: 139 tests, 1,039 expectations
  • standing review: 5/5, zero open P0-P2
  • independent milestone review: 5/5, zero open P0-P2
  • Wayfinder dogfood: 68 trails
  • package pack checks: passed

Risk and rollout

The new derivation fields are additive and earlier artifacts remain parseable. Derived identifier evidence is review-only; it does not become an automatic rewrite. Malformed lock evidence now fails loudly, which is intentional because preserve inventory is a safety boundary.

@linear-code

linear-code Bot commented Jul 14, 2026

Copy link
Copy Markdown

TRL-1134

galligan commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • queue:merge - adds this PR to the back of the merge queue
  • queue:priority - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

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.

1 participant