fix: Add missing download_url for what-if#2204
fix: Add missing download_url for what-if#2204DevAbdullah90 wants to merge 4 commits intogithub:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Adds the missing download_url for the community “What-if Analysis” extension entry so it can be installed via the CLI, and updates the community catalog metadata timestamp.
Changes:
- Bump top-level
updated_atin the community extensions catalog. - Add
download_urlto thewhatifextension entry.
Show a summary per file
| File | Description |
|---|---|
| extensions/catalog.community.json | Updates catalog timestamp and adds missing download_url for the whatif extension. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
extensions/catalog.community.json
Outdated
| "description": "Preview the downstream impact (complexity, effort, tasks, risks) of requirement changes before committing to them.", | ||
| "author": "DevAbdullah90", | ||
| "version": "1.0.0", | ||
| "download_url": "https://github.com/DevAbdullah90/spec-kit-whatif/archive/refs/heads/main.zip", |
There was a problem hiding this comment.
download_url points to the main branch archive (refs/heads/main.zip), which is mutable and can drift from the declared version: 1.0.0. Per the extension distribution decision, prefer a release/tag-pinned ZIP (e.g., a GitHub release asset or archive/refs/tags/v1.0.0.zip) so installs are reproducible and match the catalog version.
| "download_url": "https://github.com/DevAbdullah90/spec-kit-whatif/archive/refs/heads/main.zip", | |
| "download_url": "https://github.com/DevAbdullah90/spec-kit-whatif/archive/refs/tags/v1.0.0.zip", |
mnriem
left a comment
mnriem
left a comment
There was a problem hiding this comment.
Please address Copilot feedback
|
Hi @mnriem, I have addressed the final technical feedback:
Everything should now be ready for a final merge. Thanks for the guidance throughout this process! |
There was a problem hiding this comment.
Pull request overview
Adds the missing download_url for the community “What-if Analysis” extension so it can be installed via the CLI, and updates the community catalog timestamp accordingly.
Changes:
- Bumped
extensions/catalog.community.jsontop-levelupdated_at. - Added a pinned tag ZIP
download_urlfor thewhatifextension entry.
Show a summary per file
| File | Description |
|---|---|
| extensions/catalog.community.json | Updates catalog timestamp and adds the required download_url for the whatif extension to enable CLI installation. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0 new
|
Hi @mnriem, I have resolved the merge conflict in Thanks! |
There was a problem hiding this comment.
Pull request overview
This PR completes the community catalog metadata for the previously-registered “What-if Analysis” extension by adding the missing download_url so it can be installed via the CLI.
Changes:
- Add a
download_urlfield for thewhatifextension entry inextensions/catalog.community.json.
Show a summary per file
| File | Description |
|---|---|
extensions/catalog.community.json |
Adds the missing download_url for the whatif extension so CLI installs can fetch the ZIP. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
| "author": "DevAbdullah90", | ||
| "version": "1.0.0", | ||
| "download_url": "https://github.com/DevAbdullah90/spec-kit-whatif/archive/refs/tags/v1.0.0.zip", | ||
| "repository": "https://github.com/DevAbdullah90/spec-kit-whatif", |
There was a problem hiding this comment.
Since this PR modifies the whatif catalog entry, the extension’s own updated_at (inside the whatif object) and the catalog’s top-level updated_at should be bumped to the current timestamp per the publishing guide. This helps downstream consumers detect catalog/entry changes even when the extension version stays the same.
|
Hi @mnriem, I have bumped both the top-level and extension-specific Ready for the final merge! |
There was a problem hiding this comment.
Pull request overview
Adds the missing download_url to the community catalog entry for the What-if Analysis extension so it can be installed via the CLI, and updates catalog timestamps accordingly.
Changes:
- Bump
extensions/catalog.community.jsontop-levelupdated_at. - Add
download_urlfor thewhatifextension and bump that extension entry’supdated_at.
Show a summary per file
| File | Description |
|---|---|
extensions/catalog.community.json |
Adds the missing download_url for whatif and refreshes timestamps to reflect the catalog update. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 0 new
This is a small follow-up to #2182.
The previous PR was merged just before the
download_urlcould be included. This PR adds the missing download link for the "What-if Analysis" extension to ensure it can be installed via the CLI, and bumps the catalog'supdated_attimestamp.Testing
uv run specify --helpuv sync && uv run pytestAI Disclosure