feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

feat(fragments): add command with open-changelist append + per-branch impact caps - #2

Open
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux
Open

feat(fragments): add command with open-changelist append + per-branch impact caps#2
manuc66 wants to merge 13 commits into
mainfrom
feat/fragment-changelist-ux

Conversation

@manuc66

Copy link
Copy Markdown
Owner

Summary

Solves the "too many fragment files per MR" friction and adds a per-branch impact cap. Stacked on #1 (gates + API-surface dogfooding): the diff shrinks to this feature alone once #1 merges.

What changed

1. changesharp add — the open changelist

  • add appends a change to the most recent fragment (the open changelist), so a feature branch accumulates into a single file.
  • Trunk-safe: on the default branch (main/master) add always creates a separate file → concurrent pushes stay conflict-free.
  • Escape hatches: --separate (new file), --fragment <file> (specific target), --changelist <name> (deterministic named changelist).
  • new remains equivalent to add --separate (backward compatible).
  • The MaxImpact cap applies to add exactly as to new.

2. Per-branch impact caps (SemverPolicy.BranchMaxImpact)

{ "SemverPolicy": { "MaxImpact": "minor", "BranchMaxImpact": { "release/*": "patch" } } }
  • On a release/1.2 branch, an Added (Minor) fragment is refused at add and release (exit 3) unless --allow-major; Fixed (Patch) is fine.
  • Effective cap = min(global MaxImpact, matching branch entry). * suffix wildcard supported.

3. MCP parity

create_fragment gains separate / fragment / changelist params (same semantics as the CLI).

Tests

  • AppendFragment (append same/new section, create, dedup, changelist/fragment/separate targets, default-branch vs feature-branch via git) and BranchMaxImpact (matching/non-matching branch, gate messages): 16 new tests → 130 total, all green.
  • Baselines regenerated (cli-help.txt, mcp-tools.json, public-api.txt); the api-surface job derives minor (additions only).
  • Demo samples/maximpact-gate/run-demo.sh: 11/11.

Verified manually

  • Feature branch: 2 × add → 1 file, 2 sections.
  • main: 2 × add → 2 files (trunk-safe).
  • release/1.0 + BranchMaxImpact: release/* → patch: add --added exit 3 ("above the allowed impact (patch)"), add --fixed OK.

…e gate
- SemverPolicy.MaxImpact (default major = cap off): fragments/releases that
would force a Major bump are refused at new and release unless --allow-major
is passed. Interactive menu marks blocked categories and re-prompts.
- MCP tools expose the gates: validate_fragments apiMinLevel, perform_release
allowMajor/apiMinLevel.
- Dogfood the API Surface Gate on ChangeSharp itself: committed baselines
(tests/public-api: cli-help, mcp-tools, public-api via PublicApiGenerator),
scripts/update-public-api.sh, api-surface CI job, PublicApiBaselineTests.
- Demo workspace samples/maximpact-gate with self-verifying run-demo.sh.
- Docs updated (SemVer Rules, ApiSurfaceGate, McpIntegration, Roadmap).
- MCP create_fragment now enforces SemverPolicy.MaxImpact with an allowMajor
opt-in, matching the CLI new command (review finding).
- Regenerate mcp-tools.json baseline (create_fragment schema).
- Document that prerelease create/promote bypass the gates (pre-existing
behavior, shared with --api-min-level), that cli-help rewording is treated
conservatively as major by the api-surface heuristic, and that
scripts/update-public-api.sh requires jq (now checked up front).
…-major
- Add WorkspaceManager.GetCreateFragmentError and GetReleaseGateResult so the
CLI and MCP share the exact same gate sequence (floor then cap), removing the
CLI/MCP orchestration drift (create_fragment had been missed initially; gate
vs security precedence differed).
- Add NextVersionComputer.ImpactName; drop the duplicated CLI/MCP helpers.
- Record the explicit --allow-major decision in release output (CLI warning +
JSON warnings; MCP response text) as an audit trail.
- Extend samples/maximpact-gate/run-demo.sh with MCP e2e gate scenarios (11/11).
- Note the MaxImpact/Mappings interaction (Changed->Major + minor cap blocks
Changed) in SemVer Rules.
- Regenerate public-api.txt baseline (+ImpactName, +GetCreateFragmentError,
+GetReleaseGateResult).
- Extract RenderCategoryMenu and ApplyCategoryKey out of PromptForCategory.
- Extract BumpForImpact out of ComputeVersionWithWarning (behavior preserved,
covered by the existing SemVer tests).
@manuc66
manuc66 changed the base branch from feat/safety-gates-and-api-surface-dogfooding to mainAugust 22, 2026 19:32
@manuc66

Copy link
Copy Markdown
OwnerAuthor

Reopening to trigger CI after base retarget to main.

@manuc66manuc66 closed this Aug 22, 2026
@manuc66manuc66 reopened this Aug 22, 2026
@codecov

codecovBot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.54545% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.86%. Comparing base (07d4d25) to head (84c7a2a).
⚠️ Report is 7 commits behind head on main.

Files with missing linesPatch %Lines
ChangeSharp/WorkspaceManager.cs85.89%9 Missing and 13 partials ⚠️
ChangeSharp/NextVersionComputer.cs82.60%3 Missing and 1 partial ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #2 +/- ##
==========================================
+ Coverage 78.33% 80.86% +2.53% 
==========================================
Files 12 14 +2 Lines 1154 1406 +252 Branches 194 237 +43 ==========================================
+ Hits 904 1137 +233 - Misses 183 193 +10 - Partials 67 76 +9 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

… impact caps
- New 'add' command: appends a change to the most recent fragment (the open
changelist); on the default branch it always creates a separate file so
concurrent trunk pushes stay conflict-free. Escape hatches: --separate,
--fragment <file>, --changelist <name>. 'new' stays equivalent to
'add --separate'.
- SemverPolicy.BranchMaxImpact: per-branch impact caps (e.g. release/* -> patch)
that restrict branches to a lower impact; effective cap = min(global, branch).
Applies at add/new and release through the existing MaxImpact gates.
- MCP create_fragment gains separate/fragment/changelist params.
- AppendFragment + IsDefaultBranch in the library; 16 new tests (130 total).
- Docs (SemVer Rules, McpIntegration, Roadmap) and baselines regenerated.
@manuc66
manuc66force-pushed the feat/fragment-changelist-ux branch from 54ef819 to 33bb48aCompareAugust 22, 2026 19:49
…face gate
- changesharp validate --api-surface regenerates the CLI help, MCP tools, and
library public API in memory, checks the committed baselines (tests/public-api),
derives the impact vs origin/main, and gates the fragments - all in one step.
- Move MCP tool definitions into the library (McpToolCatalog) as a single source;
the MCP server and the surface check both use it.
- PublicSurfaceValidator (BaselinesMatch, DeriveImpact) in the library, JSON-aware
comparison, fully unit-tested.
- The api-surface CI job collapses from a multi-step bash heuristic to a single
'changesharp validate --api-surface' step.
- GenerateCliHelp renders under invariant culture so the baseline is stable.
- 15 new tests (145 total); baselines regenerated.
- Shared options become static fields; each command (init, new, add, status,
validate, release, publish, prerelease, remove) gets a dedicated builder
method, removing the 700-line long-method CodeFactor finding.
- Behavior and help output are byte-identical (baselines unchanged).
…ve helpers
- new and add now share RunFragmentAction (forceSeparate flag); the duplicate
message/category/gate flow is gone. CategoryFromFlags extracted.
- validate/release/remove extract their sub-branches (CheckApiMinLevelGate,
ShowDryRun, ShowFragmentList, RemoveAllFragments, ShowRemoveUsage,
RemoveSingleFragment) to drop the remaining long-method CodeFactor findings.
- Help output unchanged (baselines identical).
- new/add: RunFragmentAction is now a thin loop over ResolveFragmentMessage,
ChooseCategory, and TryAppendFragment (retry sentinel for interactive re-prompt).
- validate: rendering extracted into ReportValidationErrors/ReportValidationSuccess.
- release: pre-checks (ResolveDryRun, CheckApproval) and FinishRelease extracted;
the action is now an orchestrator.
- Behavior and help output unchanged (baselines identical).
Dogfoods the MaxImpact cap, the add/append command, and the MCP gates
(create_fragment / perform_release) end-to-end on every PR and push.
Sign up for freeto 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

@manuc66