Skip to content

feat: align extension with patchloom CLI v0.2.0 and improve docs#130

Merged
SebTardif merged 5 commits into
mainfrom
fix/improve-v0.2.0-alignment
Jun 18, 2026
Merged

feat: align extension with patchloom CLI v0.2.0 and improve docs#130
SebTardif merged 5 commits into
mainfrom
fix/improve-v0.2.0-alignment

Conversation

@SebTardif

Copy link
Copy Markdown
Contributor

Summary

Aligns the VS Code extension with patchloom CLI v0.2.0 and addresses documentation, code quality, and dependency issues found during a multi-perspective improvement rotation.

Changes

v0.2.0 alignment

  • Update PATCHLOOM_DOCS_URL to new docs site (patchloom.github.io/patchloom/)
  • Add patch merge Quick Action (three-way merge, new in v0.2.0)
  • Add 3 tests for the new quick action builder
  • Add MCP setup guide link to walkthrough

Code quality

  • Replace duplicate errorMessage in managedInstall.ts with shared formatError from util.ts (6 call sites updated)

Documentation

  • Add missing "Open Documentation" command to README commands table
  • Add 4 undocumented settings to README (patchloom.enable, patchloom.trace.server, patchloom.env, patchloom.managedInstall.autoUpdate)
  • Update AGENTS.md test counts across 6 files and add 2 missing test file entries

Dependencies

  • Update transitive dependencies
  • Fix markdown-it GHSA-6v5v-wf23-fmfq vulnerability (high severity)

Verification

  • All 252 tests pass
  • 91.23% line coverage (above 80% threshold)
  • VSIX packages cleanly (53 files, 368 KB)
  • npm run check passes (compile + compile-tests + unit tests + coverage + package)
  • Zero npm audit vulnerabilities

Ref #120

- Update PATCHLOOM_DOCS_URL to https://patchloom.github.io/patchloom/
- Add patch merge Quick Action (three-way merge via patchloom patch merge)
- Support --allow-conflicts flag and exit code 8 (CONFLICTS) handling
- Add 3 unit tests for buildPatchMergeQuickAction
- Update README: CLI version recommendation, Quick Actions table
- Update walkthrough with link to new MCP setup guide
- Update AGENTS.md test count and quickActions description

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
managedInstall.ts had a local errorMessage(error) helper that duplicated
formatError from util.ts but without the edge-case handling (empty
messages, toString: undefined). Replace all 6 call sites with the shared
helper.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
- Add 'Open Documentation' to the Commands table
- Add 4 missing settings: enable, trace.server, env,
  managedInstall.autoUpdate

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
npm audit fix resolves GHSA-6v5v-wf23-fmfq (markdown-it quadratic
complexity DoS in smartquotes). Zero vulnerabilities remaining.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
- Fix stale test counts across 6 files (binary 48->53,
  initializeProject 19->23, managedLifecycle 12->22,
  outputChannel 13->10, patchloomCli 23->29, quickActions 29->46)
- Add missing propertyBased.test.ts (13 tests) and
  verifyMcp.test.ts (15 tests) entries

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
@SebTardif SebTardif enabled auto-merge (squash) June 18, 2026 03:44
@SebTardif SebTardif merged commit cb58a47 into main Jun 18, 2026
20 checks passed
@SebTardif SebTardif deleted the fix/improve-v0.2.0-alignment branch June 18, 2026 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant