Skip to content

[preset] Update Inventory Alignment preset to v0.1.1 - #4494

Merged
KSchlobohm merged 3 commits into
mainfrom
update-inventory-alignment-preset-7c03ecd23f452522
Sep 9, 2026
Merged

[preset] Update Inventory Alignment preset to v0.1.1#4494
KSchlobohm merged 3 commits into
mainfrom
update-inventory-alignment-preset-7c03ecd23f452522

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Updates the existing inventory-alignment community preset entry to v0.1.1.

  • Updated the release download URL and SHA-256 digest.
  • Raised the required Spec Kit version to >=1.0.4 to support the declared speckit-inventory dependency.
  • Updated the catalog timestamps while preserving the original created_at value.
  • The existing community presets documentation row already matches the submitted name, description, counts, dependency, and repository, so no row change was necessary.

All validation checks passed: the submission format and checklists are valid; the public repository contains preset.yml and LICENSE; the GitHub README is readable and contains an accepted specify preset add --dev command; and the v0.1.1 release contains the submitted ZIP asset and matching digest.

Closes #4487

cc @Yash-Chindam

Generated by 🎨 Add Community Preset from Issue Submission for issue #4487 · 100.5 AIC · ⌖ 9.71 AIC · ⊞ 30.1K ·

Update inventory-alignment preset submitted by @Yash-Chindam:\n- presets/catalog.community.json (version, download_url, sha256, requires, timestamps)\n- docs/community/presets.md community presets table (no row changes required)\n\nCloses #4487\n\nAssisted-by: GitHub Copilot (model: gpt-5.2-codex, autonomous)\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mnriem
mnriem marked this pull request as ready for review September 9, 2026 18:12
@mnriem
mnriem self-requested a review as a code owner September 9, 2026 18:12
Copilot AI balanced review requested due to automatic review settings September 9, 2026 18:12
@mnriem

mnriem commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Data checks pass (form = manifest, tag-pinned, CI green), but holding merge on a dependency: the published preset.yml declares requires: extensions: speckit-inventory >=0.1.1, and that extension's catalog PR (#4486) hasn't been generated — its workflow run stranded on the safe_outputs contaminated-patch bug (#4493). Merging this first would publish a preset whose required extension isn't in the catalog. Order: land #4486's extension first, then this. (Let the automated review run here too before merge.)

@mnriem mnriem added the triage-must-have Verdict: high-value, important work for Spec Kit — do first label Sep 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The catalog contains internal inconsistencies (e.g., inventory-alignment still reports version 0.1.0, and canon-core version/URL are mismatched) that should be corrected before publishing.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the community preset catalog entry intended to move inventory-alignment to v0.1.1, refreshing its download metadata and compatibility requirements in presets/catalog.community.json.

Changes:

  • Updated the catalog top-level updated_at timestamp to 2026-09-09.
  • Updated inventory-alignment download URL + SHA-256 digest and raised requires.speckit_version to >=1.0.4.
  • Updated inventory-alignment entry updated_at timestamp.
File summaries
File Description
presets/catalog.community.json Updates catalog timestamps and preset entry metadata for the inventory-alignment update.
Review details

Suppressed comments (1)

presets/catalog.community.json:156

  • canon-core is bumped to 0.1.1 but its download_url still points to the v0.1.0 asset. Either revert the version bump or update the URL (and any related metadata) to the matching v0.1.1 release to avoid publishing a mismatched catalog entry.
      "version": "0.1.1",
      "description": "Adapts original Spec Kit workflow to work together with Canon extension.",
      "author": "Maxim Stupakov",
      "download_url": "https://github.com/maximiliamus/spec-kit-canon/releases/download/v0.1.0/spec-kit-canon-core-v0.1.0.zip",
      "repository": "https://github.com/maximiliamus/spec-kit-canon",
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread presets/catalog.community.json Outdated
Assisted-by: GitHub Copilot (model: GPT-5.4, autonomous)

Co-authored-by: mnriem <15701806+mnriem@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The canon-core entry reports version 0.1.1 while still downloading the v0.1.0 asset, creating an internal catalog inconsistency.

Review details

Suppressed comments (1)

Previously missed (1) — in code that hasn't changed since the last review.

presets/catalog.community.json:152

  • canon-core is now marked as version 0.1.1, but its download_url still points to the v0.1.0 asset. This makes the catalog entry internally inconsistent and will misreport which version users are installing.
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Keep canon-core at 0.1.0 so its version matches its release asset. Limit the preset update to inventory-alignment.

Assisted-by: GitHub Copilot (model: gpt-6-astra, autonomous)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@mnriem

mnriem commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Reverted the unrelated canon-core version bump to 0.1.0 in commit 7eee719. The PR now changes only the inventory-alignment entry and the catalog timestamp.

Posted on behalf of @mnriem by GitHub Copilot (model: gpt-6-astra).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The catalog update is internally consistent (version/URL/SHA/requirements) and follows the documented timestamp rules while preserving created_at.

Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@KSchlobohm KSchlobohm self-assigned this Sep 9, 2026
@KSchlobohm
KSchlobohm merged commit 86b7a01 into main Sep 9, 2026
17 checks passed
@mnriem
mnriem deleted the update-inventory-alignment-preset-7c03ecd23f452522 branch September 9, 2026 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated preset-submission triage-must-have Verdict: high-value, important work for Spec Kit — do first

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Preset]: Update Inventory Alignment (inventory-alignment) to v0.1.1

4 participants