Commit 1cf04b1
fix(docs): replace parent-directory VS Code settings paths with per-subdirectory enumeration (#732)
All six getting-started method docs and the installer agent template
used single parent-directory glob entries for
`chat.agentFilesLocations`, `chat.promptFilesLocations`, and
`chat.instructionsFilesLocations`. These entries failed to resolve
correctly in VS Code, preventing Copilot from discovering agents,
prompts, and instructions.
> Parent-directory paths like `".github/agents": true` do not match the
way VS Code resolves these settings. Per-subdirectory enumeration aligns
with the canonical `.vscode/settings.json` structure.
Each parent-directory entry was replaced with explicit per-subdirectory
entries matching the canonical configuration. The previously missing
`chat.agentSkillsLocations` setting was added to every settings block,
closing a gap in skills discovery.
### Getting-Started Method Docs
Six method docs received the same transformation, each preserving its
method-specific path prefix:
- Expanded **`chat.agentFilesLocations`** from 1 parent-directory entry
to 9 subdirectory entries: *ado*, *data-science*, *design-thinking*,
*github*, *installer*, *project-planning*, *hve-core*,
*hve-core/subagents*, *security-planning*
- Expanded **`chat.promptFilesLocations`** from 1 entry to 5
subdirectory entries: *ado*, *design-thinking*, *github*, *hve-core*,
*security-planning*
- Expanded **`chat.instructionsFilesLocations`** from 1 entry to 6
subdirectory entries: *ado*, *coding-standards*, *design-thinking*,
*github*, *hve-core*, *shared*
- Added **`chat.agentSkillsLocations`** with `.github/skills` and
`.github/skills/shared` entries
Files updated: *codespaces.md*, *git-ignored.md*, *mounted.md*,
*multi-root.md*, *peer-clone.md*, *submodule.md*. Local project override
entries in *submodule.md*, *codespaces.md*, and *multi-root.md* were
preserved unchanged.
### Installer Agent Template
- Added **`chat.agentSkillsLocations`** to both the generic `<PREFIX>`
template block and the devcontainer example block in
*hve-core-installer.agent.md*
- Existing per-subdirectory entries for the other three settings were
already correct and left unchanged
## Related Issue(s)
Fixes #710
## Type of Change
Select all that apply:
**Code & Documentation:**
- [x] Bug fix (non-breaking change fixing an issue)
- [ ] New feature (non-breaking change adding functionality)
- [ ] Breaking change (fix or feature causing existing functionality to
change)
- [x] Documentation update
**Infrastructure & Configuration:**
- [ ] GitHub Actions workflow
- [ ] Linting configuration (markdown, PowerShell, etc.)
- [ ] Security configuration
- [ ] DevContainer configuration
- [ ] Dependency update
**AI Artifacts:**
- [ ] Reviewed contribution with `prompt-builder` agent and addressed
all feedback
- [ ] Copilot instructions (`.github/instructions/*.instructions.md`)
- [ ] Copilot prompt (`.github/prompts/*.prompt.md`)
- [x] Copilot agent (`.github/agents/*.agent.md`)
- [ ] Copilot skill (`.github/skills/*/SKILL.md`)
> **Note for AI Artifact Contributors**:
>
> - **Agents**: Research, indexing/referencing other project (using
standard VS Code GitHub Copilot/MCP tools), planning, and general
implementation agents likely already exist. Review `.github/agents/`
before creating new ones.
> - **Skills**: Must include both bash and PowerShell scripts. See
[Skills](../docs/contributing/skills.md).
> - **Model Versions**: Only contributions targeting the **latest
Anthropic and OpenAI models** will be accepted. Older model versions
(e.g., GPT-3.5, Claude 3) will be rejected.
> - See [Agents Not
Accepted](../docs/contributing/custom-agents.md#agents-not-accepted) and
[Model Version
Requirements](../docs/contributing/ai-artifacts-common.md#model-version-requirements).
**Other:**
- [ ] Script/automation (`.ps1`, `.sh`, `.py`)
- [ ] Other (please describe):
## Sample Prompts (for AI Artifact Contributions)
**User Request:**
Use the installer agent to set up HVE-Core in a new project. The updated
agent template now includes `chat.agentSkillsLocations` entries so
skills are discoverable after installation.
**Execution Flow:**
1. User invokes the installer agent for any supported method
(peer-clone, submodule, codespace, etc.)
2. Agent generates VS Code settings JSON that includes all four `chat.*`
location settings
3. `chat.agentSkillsLocations` entries point to `.github/skills` and
`.github/skills/shared` under the appropriate prefix
**Output Artifacts:**
Generated `.vscode/settings.json` or devcontainer customization block
containing the four location settings with per-subdirectory entries.
**Success Indicators:**
Copilot discovers agents, prompts, instructions, and skills from
HVE-Core after applying the generated settings. The
`chat.agentSkillsLocations` entries appear in the output alongside the
other three location settings.
> [!NOTE]
> Human review is recommended for agent-populated Sample Prompts
content.
For detailed contribution requirements, see:
- **Common Standards**:
[docs/contributing/ai-artifacts-common.md](../docs/contributing/ai-artifacts-common.md)
- Shared standards for XML blocks, markdown quality, RFC 2119,
validation, and testing
- **Agents**:
[docs/contributing/custom-agents.md](../docs/contributing/custom-agents.md)
- Agent configurations with tools and behavior patterns
- **Prompts**:
[docs/contributing/prompts.md](../docs/contributing/prompts.md) -
Workflow-specific guidance with template variables
- **Instructions**:
[docs/contributing/instructions.md](../docs/contributing/instructions.md)
- Technology-specific standards with glob patterns
- **Skills**:
[docs/contributing/skills.md](../docs/contributing/skills.md) - Task
execution utilities with cross-platform scripts
## Testing
Automated validation performed:
- `npm run lint:md`: Passed (zero errors across all changed files)
- Content verification: Confirmed `chat.agentSkillsLocations` present in
all 7 changed files
- Regression check: Confirmed no parent-directory glob patterns remain
in any changed file
- Cross-file consistency: Verified subdirectory lists match across all
method docs and installer agent
- Manual testing was not performed
## Checklist
### Required Checks
- [x] Documentation is updated (if applicable)
- [x] Files follow existing naming conventions
- [x] Changes are backwards compatible (if applicable)
- [ ] Tests added for new functionality (if applicable) (N/A —
documentation and template corrections, no new testable functionality)
### AI Artifact Contributions
<!-- If contributing an agent, prompt, instruction, or skill, complete
these checks -->
- [ ] Used `/prompt-analyze` to review contribution
- [ ] Addressed all feedback from `prompt-builder` review
- [ ] Verified contribution follows common standards and type-specific
requirements
### Required Automated Checks
The following validation commands must pass before merging:
- [x] Markdown linting: `npm run lint:md`
- [x] Spell checking: `npm run spell-check`
- [x] Frontmatter validation: `npm run lint:frontmatter`
- [x] Skill structure validation: `npm run validate:skills`
- [x] Link validation: `npm run lint:md-links`
- [x] PowerShell analysis: `npm run lint:ps`
- [x] Plugin freshness: `npm run plugin:generate`
## Security Considerations
<!-- 1 parent 69caa84 commit 1cf04b1
File tree
7 files changed
+369
-39
lines changed- .github/agents/installer
- docs/getting-started/methods
7 files changed
+369
-39
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
533 | 533 | | |
534 | 534 | | |
535 | 535 | | |
| 536 | + | |
| 537 | + | |
| 538 | + | |
| 539 | + | |
536 | 540 | | |
537 | 541 | | |
538 | 542 | | |
| |||
618 | 622 | | |
619 | 623 | | |
620 | 624 | | |
| 625 | + | |
| 626 | + | |
| 627 | + | |
| 628 | + | |
621 | 629 | | |
622 | 630 | | |
623 | 631 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
80 | 80 | | |
81 | 81 | | |
82 | 82 | | |
83 | | - | |
84 | | - | |
85 | | - | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
86 | 113 | | |
87 | 114 | | |
88 | 115 | | |
| |||
122 | 149 | | |
123 | 150 | | |
124 | 151 | | |
125 | | - | |
126 | | - | |
127 | | - | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
128 | 182 | | |
129 | 183 | | |
130 | 184 | | |
| |||
154 | 208 | | |
155 | 209 | | |
156 | 210 | | |
157 | | - | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
158 | 216 | | |
159 | 217 | | |
160 | 218 | | |
161 | | - | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
162 | 225 | | |
163 | 226 | | |
164 | 227 | | |
165 | | - | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
166 | 237 | | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
167 | 243 | | |
168 | 244 | | |
169 | 245 | | |
| |||
193 | 269 | | |
194 | 270 | | |
195 | 271 | | |
196 | | - | |
197 | | - | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
198 | 282 | | |
199 | 283 | | |
200 | | - | |
201 | | - | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
202 | 296 | | |
203 | 297 | | |
204 | | - | |
205 | | - | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
206 | 322 | | |
207 | 323 | | |
208 | 324 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
99 | 99 | | |
100 | 100 | | |
101 | 101 | | |
102 | | - | |
103 | | - | |
104 | | - | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
105 | 132 | | |
106 | 133 | | |
107 | 134 | | |
| |||
136 | 163 | | |
137 | 164 | | |
138 | 165 | | |
139 | | - | |
140 | | - | |
141 | | - | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
142 | 196 | | |
143 | 197 | | |
144 | 198 | | |
| |||
0 commit comments