Skip to content

Update workflow-preset to v3.0.0 - #78

Merged
bigsmartben merged 3 commits into
mainfrom
workflow-preset-release-v3.0.0
Jul 26, 2026
Merged

Update workflow-preset to v3.0.0#78
bigsmartben merged 3 commits into
mainfrom
workflow-preset-release-v3.0.0

Conversation

@bigsmartben

Copy link
Copy Markdown
Owner

Summary

Test Plan

  • python3 -m pytest tests/test_presets.py -k 'workflow_preset'

Assisted-by: GitHub Actions (autonomous)

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2c95f423e2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

id: workflow-preset
name: Workflow Preset
version: 2.0.0
version: 3.0.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Update the integration smoke test for the removed override

When .github/workflows/workflow-preset-integration.yml runs its external-release-install job for version 3.0.0, line 124 still executes test -f .specify/presets/workflow-preset/commands/speckit.implement.md. This release deliberately removes that file, so the step exits under set -e and every external-release validation fails before it can verify the core implementation command. Change the workflow to assert that the preset-owned file is absent while retaining the existing check for the generated core speckit-implement skill.

AGENTS.md reference: presets/workflow-preset/AGENTS.md:L19-L20

Useful? React with 👍 / 👎.

Sync the immutable v3.0.0 Release snapshot, remove the preset-owned Implement override, preserve Tasks final Code Review, and validate Core Implement ownership across bundled and external installs.

Assisted-by: OpenAI Codex (model: GPT-5, autonomous)
Combine the release workflow's generated snapshot commit with the full issue #30 integration and validation changes.

Assisted-by: OpenAI Codex (model: GPT-5, autonomous)

# Conflicts:
#	presets/catalog.community.json
#	presets/catalog.json
@bigsmartben

Copy link
Copy Markdown
Owner Author

Integration update pushed at commit 3e64f90863510b702c7a5251e437b053656ab8bf.

Changes include the immutable v3.0.0 Release snapshot and manifest, catalog source/SHA metadata, removal of the preset-owned Implement command, Core Implement SHA preservation checks, final Tasks Code Review coverage, and updated community smoke/documentation.

Validation: Release manifest 42/42 file hashes matched; bundled and --from v3.0.0 installs matched; Core Implement SHA remained eb0da8be9513b03871998f5c20371694de1471c1172084722f72ca97535e9a91; preset contracts passed 57 tests; targeted fork tests passed 7 tests; full fork suite passed 3891 tests with 88 skipped.

Posted on behalf of @bigsmartben by OpenAI Codex (model: GPT-5, autonomous).

@bigsmartben
bigsmartben merged commit b7af9ef into main Jul 26, 2026
15 checks passed
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.

1 participant