feat: align extension with patchloom CLI v0.2.0 and improve docs#130
Merged
Conversation
- 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>
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
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.
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
PATCHLOOM_DOCS_URLto new docs site (patchloom.github.io/patchloom/)patch mergeQuick Action (three-way merge, new in v0.2.0)Code quality
errorMessageinmanagedInstall.tswith sharedformatErrorfromutil.ts(6 call sites updated)Documentation
patchloom.enable,patchloom.trace.server,patchloom.env,patchloom.managedInstall.autoUpdate)Dependencies
Verification
npm run checkpasses (compile + compile-tests + unit tests + coverage + package)Ref #120