Skip to content

Conversation

@CalvinAllen
Copy link
Contributor

Summary

  • Remove VsixTemplateZip item type and related zipping targets (VSSDK handles template packaging natively when Content entries are in manifest)
  • Keep VsixTemplateReference for cross-project templates (copies folders from referenced projects to local ProjectTemplates/ItemTemplates folders)
  • Add proper VSIXSubPath metadata so copied templates are placed at correct paths in the VSIX
  • Update E2E tests: remove PreBuiltZip test, add Reference and ReferenceManual tests
  • Update workflow to verify new template reference tests
  • Update documentation to reflect simplified template handling

Test plan

  • Build E2E.Templates.Reference test - templates copied from SharedTemplates and included in VSIX
  • Build E2E.Templates.ReferenceManual test - same functionality
  • Verify VSIX contents have templates at correct paths (ProjectTemplates/ProjectTemplate/, ItemTemplates/ItemTemplate/)
  • CI workflow passes all E2E tests

Closes #39

- Remove VsixTemplateZip item type and related targets (VSSDK handles
  template packaging natively when Content entries are in manifest)
- Keep VsixTemplateReference for cross-project templates (copies folders
  from referenced projects to local ProjectTemplates/ItemTemplates)
- Add proper VSIXSubPath metadata so copied templates are placed at
  correct paths in the VSIX
- Update E2E tests: remove PreBuiltZip, add Reference and ReferenceManual
- Update workflow to verify new template reference tests
- Update documentation to reflect simplified template handling

Closes #39
Add Content items for auto-discovered project and item templates so they
are included in the VSIX package. The removal of template zipping also
removed the mechanism that added templates to the VSIX, which broke
auto-discovery scenarios.
Update documentation to accurately reflect that this SDK includes
template files in the VSIX, not VSSDK. The manifest Content entries
are for VS template registration, not file inclusion.
@CalvinAllen CalvinAllen merged commit 410bde4 into main Jan 7, 2026
3 checks passed
@CalvinAllen CalvinAllen deleted the chore/sdk/remove-unused-template-zipping branch January 7, 2026 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

chore(sdk): remove unused template zipping code

2 participants