Sync agentic harness from IMS: enforcement gates, tested hooks, memories, licensing - #231
Merged
Merged
Conversation
…ies, licensing Ports the harness enforcement layer NetPace was missing, adapted to its single-src/ layout (tests live under src/, not a top-level tests/) and NETPACE_* overrides. Gates: - green-gate.sh — deny `dotnet test --no-build` when it would run a stale/absent build (promotes feedback_dotnet_test_no_build). Wired as a PreToolUse(Bash) hook. - traceability-gate.sh — enforce Constitution §VIII deterministically: spec.md **Scenario:** ⟷ test-plan.md #### Scenario: bijection, and #### Scenario: → test // SCENARIO: coverage under src/. Wired as the Stop hook. NetPace already uses // SCENARIO: markers (9 test files), so edge-2 is live, not latent. - no-skipped-tests.sh — fold IMS improvements into the existing gate: xUnit v3 SkipUnless=/SkipWhen=, a --check audit mode immune to the env override, and CLAUDE_PROJECT_DIR rooting. Hook hygiene: - A runnable .tests.sh matrix for each gate (52/12/34 cases, all green) and a hooks/README.md documenting the three gates, their overrides, and scope. - statusline-command.sh + settings.json statusLine/preferredNotifChannel (terminal bell). Memories (project-agnostic, adapted to NetPace): - feedback_dotnet_test_no_build, feedback_rerun_tests_before_done, feedback_markdown_soft_wrap, feedback_plain_language_decisions, feedback_audit_class_after_two_failures (+ MEMORY.md index lines). Constitution amendment (bundled by request): - §VI Minimal Dependencies gains a permissive-licence constraint: runtime deps MUST be MIT/Apache-2.0/BSD; copyleft prohibited without justification + maintainer sign-off. 1.5.0 → 1.6.0 (MINOR). Sync Impact Report updated in-file. Verification: all three gate matrices pass; each gate --checks clean against the tree; settings.json validates. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T6dd1XBNhEq42D6EsY6nPz
FrankRay78
added a commit
that referenced
this pull request
Jul 30, 2026
Aligns NetPace's workflow docs with IMS's generic/delta split:
- Add docs/agentic-workflow.md — the generic, stack-portable guide, re-synced
from IMS's current version (body byte-identical; only the provenance comment
differs, reflecting NetPace as the source-of-truth owner).
- Add docs/agentic-workflow-NetPace.md — the NetPace delta ("sidekick"),
modelled on IMS's agentic-workflow-IMS.md. Records only where NetPace differs
from the generic guide: has CI (5 workflows incl. Review B via claude.yml),
a release pipeline (NuGet + cross-platform binaries), Change-Intent-Records
as its decision ledger, no surface tiers, the gates from #231 (green-gate,
no-skipped, traceability + format-on-commit + PR pre-flight; no stack-guard,
no UI-automation denylist), and console-output snapshot verification.
Flags one unreconciled divergence: NetPace formats on commit, which the
generic guide now argues against.
- Delete docs/agentic-software-development-workflow.md — the April 2026 version,
~3 months stale and superseded by the two files above. No references to it
remain anywhere in the repo.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T6dd1XBNhEq42D6EsY6nPz
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
A cross-repo harness audit against IMS (the more built-out sibling) found NetPace was missing the enforcement layer for policies it already declares, plus one .NET-generic guard. This PR closes that gap. Everything is adapted to NetPace's single-
src/layout (tests live undersrc/, not a top-leveltests/) andNETPACE_*overrides.Most of IMS's extra harness is legitimately IMS-only (Fabric stack-guard, UI-automation denylist, surface tiers, domain memories) and is deliberately not ported.
What
Enforcement gates
green-gate.sh(new) — deniesdotnet test --no-buildwhen it would run a stale or absent build. Pure .NET footgun that was previously unguarded. Wired as aPreToolUse(Bash)hook.traceability-gate.sh(new) — enforces Constitution §VIII deterministically:spec.md **Scenario:**⟷test-plan.md #### Scenario:bijection, and#### Scenario:→ test// SCENARIO:coverage undersrc/. NetPace already declared §VIII but nothing enforced it. NetPace already uses// SCENARIO:markers (9 test files), so the coverage edge is live, not latent. Wired as theStophook.no-skipped-tests.sh(updated) — folds IMS's improvements into the existing gate: xUnit-v3SkipUnless=/SkipWhen=, a--checkaudit mode immune to the env override, andCLAUDE_PROJECT_DIRrooting.Hook hygiene
*.tests.shmatrix for each gate (52 / 12 / 34 cases, all green) and ahooks/README.mddocumenting each gate, its override, and its scope. Hooks are now tested and documented, not just present.statusline-command.sh+settings.jsonstatusLine/preferredNotifChannel(terminal bell).Memories (project-agnostic, adapted)
feedback_dotnet_test_no_build,feedback_rerun_tests_before_done,feedback_markdown_soft_wrap,feedback_plain_language_decisions,feedback_audit_class_after_two_failures(+MEMORY.mdindex).Constitution amendment — bundled in this PR (flagged deliberately)
§VI Minimal Dependenciesgains a permissive-licence constraint: runtime dependencies MUST be MIT/Apache-2.0/BSD; copyleft (GPL/LGPL/AGPL) prohibited without documented justification and maintainer sign-off. 1.5.0 → 1.6.0 (MINOR). Sync Impact Report updated in-file.Verification
--checks clean against the current tree (steady state = no-op).settings.jsonvalidates as JSON.🤖 Generated with Claude Code