From 8ea814850d3e4f3c9a52a12cf8e7a07fede7fbef Mon Sep 17 00:00:00 2001 From: t Date: Sun, 25 Oct 2026 08:19:41 +0100 Subject: [PATCH 1/2] chore(roadmap): entries for PMAT-5446, PMAT-5909 and PMAT-5803 so #6294 and #6309 split by ticket PMAT-5446 covers the #5893 planted-.pv tests carried by #6309 (issue #5446: slow contracts-cli ont4 gate tests); PMAT-5909 and PMAT-5803 take their scope from #6294. Entries plus the regenerated aggregate only. PMAT-Ticket: PMAT-5446 Co-Authored-By: Claude Sonnet 5.5 --- docs/roadmaps/entries/PMAT-5446.yaml | 22 +++++++++ docs/roadmaps/entries/PMAT-5803.yaml | 23 ++++++++++ docs/roadmaps/entries/PMAT-5909.yaml | 23 ++++++++++ docs/roadmaps/roadmap.yaml | 68 ++++++++++++++++++++++++++++ 4 files changed, 136 insertions(+) create mode 100644 docs/roadmaps/entries/PMAT-5446.yaml create mode 100644 docs/roadmaps/entries/PMAT-5803.yaml create mode 100644 docs/roadmaps/entries/PMAT-5909.yaml diff --git a/docs/roadmaps/entries/PMAT-5446.yaml b/docs/roadmaps/entries/PMAT-5446.yaml new file mode 100644 index 0000000000..50606c00b7 --- /dev/null +++ b/docs/roadmaps/entries/PMAT-5446.yaml @@ -0,0 +1,22 @@ +- id: PMAT-5446 + github_issue: 5446 + item_type: task + title: 'ci: shrink the slow quick-tier tests (apr-cli test_llm_band, contracts-cli ont4 gate tests) at the root' + status: in_progress + priority: medium + assigned_to: null + created: 2026-10-25T07:50:00Z + updated: 2026-10-25T07:50:00Z + spec: null + acceptance_criteria: + - 'contracts-cli ont4_relations_gate, ont4b_shapes_gate and ont5_consistency_gate lint a planted fixture or copied files, not the full contracts tree, and each finishes well under 60 s (landed by #5893 via the EMR-001 integration PR #6309)' + - the pv-sat corpus() lint tests use a planted .pv fixture (same change) + - apr-cli commands::test_llm_band tests no longer take 60-240 s each (separate PR; not covered by #6309) + phases: [] + subtasks: [] + estimated_effort: null + labels: + - kind:code + notes: |- + Entry minted for the #5893 planted-.pv tests carried by #6309 (quorum handoff triage-quorum-tickets-6294-6309-0718Z). Issue #5446 names both slow families; #5893 is the contracts-cli half (gate tests lint a fixture, 11/11 per its body). The test_llm_band half has no PR yet, so the item stays in_progress. + epic: null diff --git a/docs/roadmaps/entries/PMAT-5803.yaml b/docs/roadmaps/entries/PMAT-5803.yaml new file mode 100644 index 0000000000..122a4e256e --- /dev/null +++ b/docs/roadmaps/entries/PMAT-5803.yaml @@ -0,0 +1,23 @@ +- id: PMAT-5803 + github_issue: 5803 + item_type: task + title: 'build: FLOW-010 cut 1 — move the tuner tests that need aprender-core out of aprender-compute so compute drops its dev-dependency on core' + status: in_progress + priority: medium + assigned_to: null + created: 2026-10-25T07:50:00Z + updated: 2026-10-25T07:50:00Z + spec: null + acceptance_criteria: + - the tuner training-data-quality tests move from crates/aprender-compute/tests/tuner_falsification to crates/aprender-core/tests/tuner_training_data_quality.rs and still run (ci/explicit-test-commands.d fragments 295 and 298 name them) + - aprender-compute has no dev-dependency on aprender-core + - scripts/check_tuner_test_move.sh proves the move lost no test (count and names before and after) + - one PR, at most 400 lines or a recorded PMAT-Size-Waiver (the move was 810 lines in #6294 commit ebf773587a, waiver sanctioned by quorum triage-quorum-door6261-conditions-2136Z) + phases: [] + subtasks: [] + estimated_effort: null + labels: + - kind:code + notes: |- + Scope taken from #6294 (commit 2, PMAT-5803; issue #5803, FLOW-010 five-whys on crate fan-out, cut 1). Entry minted so w5032 can cut this piece as its own PR. + epic: null diff --git a/docs/roadmaps/entries/PMAT-5909.yaml b/docs/roadmaps/entries/PMAT-5909.yaml new file mode 100644 index 0000000000..2563051515 --- /dev/null +++ b/docs/roadmaps/entries/PMAT-5909.yaml @@ -0,0 +1,23 @@ +- id: PMAT-5909 + github_issue: 5909 + item_type: task + title: 'build: split env_key out of aprender-contracts into aprender-contracts-key so a contracts edit stops dirtying 47 of 85 crates' + status: in_progress + priority: medium + assigned_to: null + created: 2026-10-25T07:50:00Z + updated: 2026-10-25T07:50:00Z + spec: null + acceptance_criteria: + - a new crate crates/aprender-contracts-key holds env_key (src/lib.rs, README, Cargo.toml) and aprender-contracts re-uses it through build_helper + - the build.rs and Cargo.toml of each dependent crate (aprender-core, -compute, -db, -orchestrate, -present-cli, -present-core, -rag, -serve, -simulate, -train, -viz-ttop) depend on aprender-contracts-key instead of aprender-contracts + - cargo metadata --locked is clean and the crate is registered in scripts/release/publish-order.txt and scripts/cascade-publish.sh; publishing it stays on the release path + - one PR, at most 400 lines, no Sensitive-path change beyond the workspace member and publish order lines (cut from #6294 commit 4b8f637715) + phases: [] + subtasks: [] + estimated_effort: null + labels: + - kind:code + notes: |- + Scope taken from #6294 (commit 1, PMAT-5909; issue #5909: aprender-contracts is 24 commits in 30 days and dirties 47 of 85 crates). #6294 is split by quorum triage-quorum-tickets-6294-6309-0718Z into one PR per ticket; this entry lets w5032 cut this piece. + epic: null diff --git a/docs/roadmaps/roadmap.yaml b/docs/roadmaps/roadmap.yaml index b5bd1bc998..5f1bb142f2 100644 --- a/docs/roadmaps/roadmap.yaml +++ b/docs/roadmaps/roadmap.yaml @@ -25319,6 +25319,28 @@ Not done, pending the ruling: moving the 31 fixture files to an obviously synthe - kind:code notes: null epic: 4780 +- id: PMAT-5446 + github_issue: 5446 + item_type: task + title: 'ci: shrink the slow quick-tier tests (apr-cli test_llm_band, contracts-cli ont4 gate tests) at the root' + status: in_progress + priority: medium + assigned_to: null + created: 2026-10-25T07:50:00Z + updated: 2026-10-25T07:50:00Z + spec: null + acceptance_criteria: + - 'contracts-cli ont4_relations_gate, ont4b_shapes_gate and ont5_consistency_gate lint a planted fixture or copied files, not the full contracts tree, and each finishes well under 60 s (landed by #5893 via the EMR-001 integration PR #6309)' + - the pv-sat corpus() lint tests use a planted .pv fixture (same change) + - apr-cli commands::test_llm_band tests no longer take 60-240 s each (separate PR; not covered by #6309) + phases: [] + subtasks: [] + estimated_effort: null + labels: + - kind:code + notes: |- + Entry minted for the #5893 planted-.pv tests carried by #6309 (quorum handoff triage-quorum-tickets-6294-6309-0718Z). Issue #5446 names both slow families; #5893 is the contracts-cli half (gate tests lint a fixture, 11/11 per its body). The test_llm_band half has no PR yet, so the item stays in_progress. + epic: null - id: PMAT-5463 github_issue: 5463 item_type: task @@ -25375,6 +25397,29 @@ Not done, pending the ruling: moving the 31 fixture files to an obviously synthe - kind:code notes: null epic: 3863 +- id: PMAT-5803 + github_issue: 5803 + item_type: task + title: 'build: FLOW-010 cut 1 — move the tuner tests that need aprender-core out of aprender-compute so compute drops its dev-dependency on core' + status: in_progress + priority: medium + assigned_to: null + created: 2026-10-25T07:50:00Z + updated: 2026-10-25T07:50:00Z + spec: null + acceptance_criteria: + - the tuner training-data-quality tests move from crates/aprender-compute/tests/tuner_falsification to crates/aprender-core/tests/tuner_training_data_quality.rs and still run (ci/explicit-test-commands.d fragments 295 and 298 name them) + - aprender-compute has no dev-dependency on aprender-core + - scripts/check_tuner_test_move.sh proves the move lost no test (count and names before and after) + - one PR, at most 400 lines or a recorded PMAT-Size-Waiver (the move was 810 lines in #6294 commit ebf773587a, waiver sanctioned by quorum triage-quorum-door6261-conditions-2136Z) + phases: [] + subtasks: [] + estimated_effort: null + labels: + - kind:code + notes: |- + Scope taken from #6294 (commit 2, PMAT-5803; issue #5803, FLOW-010 five-whys on crate fan-out, cut 1). Entry minted so w5032 can cut this piece as its own PR. + epic: null - id: PMAT-5817 github_issue: 5817 item_type: task @@ -25550,6 +25595,29 @@ Not done, pending the ruling: moving the 31 fixture files to an obviously synthe - kind:measurement notes: null epic: 5510 +- id: PMAT-5909 + github_issue: 5909 + item_type: task + title: 'build: split env_key out of aprender-contracts into aprender-contracts-key so a contracts edit stops dirtying 47 of 85 crates' + status: in_progress + priority: medium + assigned_to: null + created: 2026-10-25T07:50:00Z + updated: 2026-10-25T07:50:00Z + spec: null + acceptance_criteria: + - a new crate crates/aprender-contracts-key holds env_key (src/lib.rs, README, Cargo.toml) and aprender-contracts re-uses it through build_helper + - the build.rs and Cargo.toml of each dependent crate (aprender-core, -compute, -db, -orchestrate, -present-cli, -present-core, -rag, -serve, -simulate, -train, -viz-ttop) depend on aprender-contracts-key instead of aprender-contracts + - cargo metadata --locked is clean and the crate is registered in scripts/release/publish-order.txt and scripts/cascade-publish.sh; publishing it stays on the release path + - one PR, at most 400 lines, no Sensitive-path change beyond the workspace member and publish order lines (cut from #6294 commit 4b8f637715) + phases: [] + subtasks: [] + estimated_effort: null + labels: + - kind:code + notes: |- + Scope taken from #6294 (commit 1, PMAT-5909; issue #5909: aprender-contracts is 24 commits in 30 days and dirties 47 of 85 crates). #6294 is split by quorum triage-quorum-tickets-6294-6309-0718Z into one PR per ticket; this entry lets w5032 cut this piece. + epic: null - id: PMAT-5914 github_issue: 5914 item_type: task From 715bcd4b06183ae0b298f747f6ebb6aec125357b Mon Sep 17 00:00:00 2001 From: t Date: Sun, 25 Oct 2026 08:54:58 +0100 Subject: [PATCH 2/2] docs(roadmap): PMAT-5909 criterion 1 states what the code does: aprender-contracts keeps its own env_key.rs, the key crate is a dev-dependency (Refs #5909) PMAT-Ticket: PMAT-5909 Co-Authored-By: Claude Sonnet 5.5 --- docs/roadmaps/entries/PMAT-5909.yaml | 2 +- docs/roadmaps/roadmap.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/roadmaps/entries/PMAT-5909.yaml b/docs/roadmaps/entries/PMAT-5909.yaml index 2563051515..2c550c0768 100644 --- a/docs/roadmaps/entries/PMAT-5909.yaml +++ b/docs/roadmaps/entries/PMAT-5909.yaml @@ -9,7 +9,7 @@ updated: 2026-10-25T07:50:00Z spec: null acceptance_criteria: - - a new crate crates/aprender-contracts-key holds env_key (src/lib.rs, README, Cargo.toml) and aprender-contracts re-uses it through build_helper + - a new crate crates/aprender-contracts-key holds env_key (src/lib.rs, README, Cargo.toml) and aprender-contracts keeps its own src/env_key.rs (included by build.rs and build_helper.rs) and depends on the key crate as a dev-dependency only, where a test asserts the two builders agree - the build.rs and Cargo.toml of each dependent crate (aprender-core, -compute, -db, -orchestrate, -present-cli, -present-core, -rag, -serve, -simulate, -train, -viz-ttop) depend on aprender-contracts-key instead of aprender-contracts - cargo metadata --locked is clean and the crate is registered in scripts/release/publish-order.txt and scripts/cascade-publish.sh; publishing it stays on the release path - one PR, at most 400 lines, no Sensitive-path change beyond the workspace member and publish order lines (cut from #6294 commit 4b8f637715) diff --git a/docs/roadmaps/roadmap.yaml b/docs/roadmaps/roadmap.yaml index 5f1bb142f2..571b77a054 100644 --- a/docs/roadmaps/roadmap.yaml +++ b/docs/roadmaps/roadmap.yaml @@ -25606,7 +25606,7 @@ Not done, pending the ruling: moving the 31 fixture files to an obviously synthe updated: 2026-10-25T07:50:00Z spec: null acceptance_criteria: - - a new crate crates/aprender-contracts-key holds env_key (src/lib.rs, README, Cargo.toml) and aprender-contracts re-uses it through build_helper + - a new crate crates/aprender-contracts-key holds env_key (src/lib.rs, README, Cargo.toml) and aprender-contracts keeps its own src/env_key.rs (included by build.rs and build_helper.rs) and depends on the key crate as a dev-dependency only, where a test asserts the two builders agree - the build.rs and Cargo.toml of each dependent crate (aprender-core, -compute, -db, -orchestrate, -present-cli, -present-core, -rag, -serve, -simulate, -train, -viz-ttop) depend on aprender-contracts-key instead of aprender-contracts - cargo metadata --locked is clean and the crate is registered in scripts/release/publish-order.txt and scripts/cascade-publish.sh; publishing it stays on the release path - one PR, at most 400 lines, no Sensitive-path change beyond the workspace member and publish order lines (cut from #6294 commit 4b8f637715)