Skip to content

Add missing directories to test-vault-populated fixture#3

Merged
p3ob7o merged 2 commits into
claude/review-assistant-requirements-Pxq4ffrom
copilot/sub-pr-1
Feb 1, 2026
Merged

Add missing directories to test-vault-populated fixture#3
p3ob7o merged 2 commits into
claude/review-assistant-requirements-Pxq4ffrom
copilot/sub-pr-1

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 1, 2026

The test-vault-populated fixture was incomplete, lacking 4 of the 11 required top-level directories per the vault structure spec. This would cause T01.7 to fail once validate_vault_structure is implemented.

Changes

  • Added missing directories: .claude/, 2_Drafts/, 5_Resources/, 7_Assets/
  • Added .gitkeep files to track empty directories in git

The fixture now matches the complete vault structure defined in F01-vault-structure.md (10 numbered folders + .claude).


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: p3ob7o <1436372+p3ob7o@users.noreply.github.com>
Copilot AI changed the title [WIP] Update vault structure tests based on feedback Add missing directories to test-vault-populated fixture Feb 1, 2026
Copilot AI requested a review from p3ob7o February 1, 2026 17:30
@p3ob7o p3ob7o marked this pull request as ready for review February 1, 2026 19:38
@p3ob7o p3ob7o merged commit 8637acc into claude/review-assistant-requirements-Pxq4f Feb 1, 2026
@p3ob7o p3ob7o deleted the copilot/sub-pr-1 branch February 1, 2026 19:39
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.

2 participants