-
Notifications
You must be signed in to change notification settings - Fork 126
Closed
Closed
Copy link
Labels
infrastructureRepository infrastructure and toolingRepository infrastructure and toolingmaintenanceMaintenance work, no version bumpMaintenance work, no version bump
Milestone
Description
Summary
Move collection-related scripts (Validate-Collections.ps1, Modules/PluginHelpers.psm1) from scripts/plugins/ to a new scripts/collections/ top-level folder. Collection manifests serve both plugin generation and extension packaging; their validation and shared utilities belong at a scope that reflects both consumers.
Motivation
scripts/plugins/ implies the scripts are specific to "Copilot CLI plugin generation." In practice:
Validate-Collections.ps1is referenced bynpm run lint:collections-metadata(linting scope)PluginHelpers.psm1exports functions consumed by bothGenerate-Plugins.ps1andPackage-Extension.ps1- Collection manifests in
collections/are a top-level concern, not a plugin-specific one
A scripts/collections/ directory aligns the script organization with the data it operates on.
Deliverables
- Create
scripts/collections/directory - Move
Validate-Collections.ps1toscripts/collections/Validate-Collections.ps1 - Move/refactor shared functions from
PluginHelpers.psm1intoscripts/collections/Modules/CollectionHelpers.psm1(collection-specific functions) while keeping plugin-specific functions inscripts/plugins/ - Update
Generate-Plugins.ps1to import from the new module location - Update
Package-Extension.ps1import paths - Update
package.jsonnpm scripts (lint:collections-metadata,plugin:validate) - Update Pester test paths in
scripts/tests/ - Update
scripts/README.mdandcopilot-instructions.mdproject structure section
Acceptance Criteria
-
scripts/collections/directory exists with relocated scripts -
npm run lint:collections-metadataruns from new location -
npm run plugin:generateandnpm run plugin:validatework correctly -
npm run extension:packageworks correctly - All Pester tests pass with updated paths
-
scripts/README.mdupdated - No broken imports or references
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
infrastructureRepository infrastructure and toolingRepository infrastructure and toolingmaintenanceMaintenance work, no version bumpMaintenance work, no version bump