-
Notifications
You must be signed in to change notification settings - Fork 38
fix(extension): resolve path resolution issues in Windows/WSL environments #407
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…ify extension configuration and execution contexts - add details on dual-mode configuration for workspace and UI - explain script fallback patterns for different execution scenarios 🔧 - Generated by Copilot
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #407 +/- ##
==========================================
- Coverage 60.96% 60.93% -0.04%
==========================================
Files 19 19
Lines 3233 3233
==========================================
- Hits 1971 1970 -1
- Misses 1262 1263 +1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds the extensionKind configuration to the HVE Core VS Code extension and updates documentation to explain dual-mode execution contexts. The changes aim to address file accessibility issues reported in issue #390 where instruction files and scripts were not accessible in Windows/WSL environments.
Changes:
- Added
extensionKind: ["workspace", "ui"]to package.json to enable dual-mode extension execution - Added "Extension Configuration" section to PACKAGING.md documenting the extensionKind setting
- Standardized unordered list formatting from hyphens to asterisks throughout PACKAGING.md
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| extension/package.json | Added extensionKind configuration to support both workspace and UI execution modes |
| extension/PACKAGING.md | Added documentation explaining extension execution contexts and standardized list formatting |
…GING.md - enhance descriptions for workspace and UI modes - explain access to bundled files and path resolution - detail declarative nature of the extension
Description
This PR fixes critical installation issues where GitHub Copilot cannot access instruction files and scripts in Windows/WSL environments. The changes introduce dual-mode extension configuration with bundled resources to ensure reliable file access across all VS Code execution contexts.
Backgrounder to provide resolution context
According to VS Code Remote Extensions documentation, UI Extensions cannot directly access files in the remote workspace, which causes instruction files and scripts to become unavailable.
This dual-mode configuration with bundled resources solves critical installation issues:
Instruction files not found: In Windows/WSL environments, when Copilot runs as a UI extension, it cannot access workspace files through the remote workspace path. The documented behavior states UI extensions "cannot directly access files in the remote workspace"—bundled instruction files provide the required fallback.
Cross-platform path resolution failures: Windows paths (e.g.,
/Users/username/.vscode-insiders/extensions/...) fail when referenced from WSL Linux environments. VS Code Server runs standard Node.js (not Electron) in remote contexts, requiring platform-independent access patterns.Remote workspace limitations: In Codespaces, devcontainers, and SSH hosts, workspace file access depends on extension kind. Bundling ensures consistent access regardless of where the extension host executes.
The extension prioritizes local workspace files when running in workspace mode but seamlessly falls back to bundled copies when running in UI mode or when path resolution fails across OS boundaries.
Summary of Changes
fix(extension): Added
extensionKindconfiguration to support both workspace and UI execution modesdocs(extension): Added documentation explaining extension configuration and path resolution
style(extension): Standardized list formatting from hyphens to asterisks throughout PACKAGING.md for consistency
Related Issue(s)
Fixes #390
Type of Change
Select all that apply:
Code & Documentation:
Infrastructure & Configuration:
AI Artifacts:
prompt-builderagent and addressed all feedback.github/instructions/*.instructions.md).github/prompts/*.prompt.md).github/agents/*.agent.md)Other:
.ps1,.sh,.py)Testing
Tested by reviewing the changes against VS Code's official Remote Extensions documentation and verifying that the configuration matches documented patterns for dual-mode extensions.
Checklist
Required Checks
AI Artifact Contributions
/prompt-analyzeto review contributionprompt-builderreviewRequired Automated Checks
The following validation commands must pass before merging:
npm run lint:mdnpm run spell-checknpm run lint:frontmatternpm run lint:md-linksnpm run lint:psSecurity Considerations
Additional Notes
The extension configuration change (
extensionKind: ["workspace", "ui"]) should ensure that the extension works correctly in both direct workspace access scenarios and restricted UI-only scenarios. This is particularly important for resolving issue #390 where users reported instruction files not found errors in WSL environments.