Skip to content

chore(release): add a release workflow skill - #69

Merged
sentomk merged 2 commits into
mainfrom
chore/release-skill
Oct 29, 2026
Merged

sentomk merged 2 commits into
mainfrom
chore/release-skill

Conversation

@sentomk

@sentomk sentomk commented Oct 29, 2026

Copy link
Copy Markdown
Owner

Summary

Adds a project skill that describes the release workflow used for v0.9.4 and v0.9.5, so a release can be prepared and published from one instruction. The skill is documentation for the agent; it changes no library code.

Changes

  • .claude/skills/release/SKILL.md: release PR steps (changelog, performance note with a measurement method, version bumps, registration), post-merge steps (tag, GitHub release, portfile SHA512), and the sentomk/vcpkg and microsoft/vcpkg update.
  • States the standing rules: single-commit PRs, Conventional Commits, Summary / Changes / Testing PR bodies, no Co-Authored-By trailer.

Testing

  • Docs only. The steps were checked against the v0.9.4 release PR and the merged microsoft/vcpkg PRs for patternia.
  • Commit message checked with scripts/check_commit_msg.py.

biobo-bot[bot]
biobo-bot Bot previously approved these changes Oct 29, 2026

@biobo-bot biobo-bot Bot 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.

Docs-only change, and the workflow matches the repo — approving.

Checked

  • The rules it cites are real: scripts/check_commit_msg.py enforces the Conventional Commits subject (type list, imperative, lowercase, no trailing period), and .github/workflows/validate.yml checks for the bold **Summary** / **Changes** / **Testing** sections.
  • The files and options it names exist: PTN_BUILD_TESTS in CMakeLists.txt, the version in project(... VERSION ...), and packaging/vcpkg/portfile.cmake / vcpkg.json. The note about keeping upstream's REPO sentomk/patternia casing makes sense, since the in-repo portfile uses SentoMK/patternia.
  • Phase 1 mirrors what the v0.9.5 release PR did (release note, perf note, page registration in three places, version bumps), and keeping the portfile SHA512 out of the release PR is the right call since it needs the tag.

One non-blocking nit (inline): the vcpkg fork step describes the commit order twice in ways that read as contradictory. Details on the line.

Comment thread .claude/skills/release/SKILL.md Resolved
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Problem
- Releasing needs a fixed sequence across the repo, GitHub and two
  vcpkg repositories, and the details live only in past PRs.

Implementation
- Add .claude/skills/release/SKILL.md covering the release PR, the
  performance measurement, tagging, the GitHub release, the portfile
  SHA512 and the sentomk/vcpkg and microsoft/vcpkg updates.
- Record the commit and PR rules the workflow must follow, including
  no Co-Authored-By trailer.

Tests
- Docs only; steps follow the v0.9.4 release and the merged
  microsoft/vcpkg PRs for patternia.

@biobo-bot biobo-bot Bot 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.

Re-reviewed the force-push to 0da25e9. The only change from eb3555a is Phase 1 step 1: the release branch is now release/X.Y.Z (no v) instead of chore/release-vX.Y.Z. That's consistent with the rest of the skill (the tag and patternia-X.Y.Z fork branch carry their own prefixes) and doesn't affect any other step.

The earlier non-blocking nit about the x-add-version / commit order in Phase 2 step 4 still applies, but it isn't a blocker. LGTM.

@github-actions

github-actions Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Benchmark comparison (informational)

Base 0a0650a vs PR ceb0db2, run one after the other on the same GitHub-hosted runner. This does not block merging.

Noise estimate: the 20 control rows below do not run Patternia code, so they should not change between base and PR. As a group they moved by a median of +0.0% (run order and machine drift); that offset is divided out of the Patternia deltas (Adjusted). The controls then scatter by a median of 0.4%. Adjusted differences under 5.0% are treated as noise.

Patternia: 8 benchmarks, 0 slower, 0 faster, 8 within noise.

Benchmark Base (ns) PR (ns) Delta Adjusted Verdict
PatterniaPipe CommandParser 1.599 1.636 +2.3% +2.3% within noise
PatterniaPipe PacketMixed 1.414 1.411 -0.2% -0.2% within noise
PatterniaPipe PacketMixedHeavyBind 1.522 1.520 -0.1% -0.2% within noise
PatterniaPipe ProtocolRouter 1.318 1.314 -0.2% -0.3% within noise
PatterniaPipe VariantAltHot 1.366 1.366 -0.0% -0.0% within noise
PatterniaPipe VariantAltHotGuarded 1.501 1.502 +0.1% +0.1% within noise
PatterniaPipe VariantMixed 1.657 1.651 -0.4% -0.4% within noise
PatterniaPipe VariantMixedGuarded 1.566 1.551 -0.9% -0.9% within noise
Control rows (not Patternia code)
Benchmark Base (ns) PR (ns) Delta
IfElse CommandParser 1.446 1.457 +0.8%
IfElse ProtocolRouter 1.740 1.722 -1.0%
Sequential VariantAltHot 1.801 2.323 +29.0%
Sequential VariantAltHotGuarded 2.323 2.348 +1.1%
Sequential VariantMixed 0.907 0.909 +0.3%
Sequential VariantMixedGuarded 0.944 0.948 +0.5%
StdVisit CommandParser 1.103 1.103 -0.0%
StdVisit ProtocolRouter 1.314 1.305 -0.7%
StdVisit VariantAltHot 2.146 1.778 -17.2%
StdVisit VariantAltHotGuarded 1.610 1.732 +7.5%
StdVisit VariantMixed 1.832 2.471 +34.9%
StdVisit VariantMixedGuarded 1.792 1.789 -0.2%
SwitchIndex VariantAltHot 2.324 2.320 -0.1%
SwitchIndex VariantAltHotGuarded 2.461 2.456 -0.2%
SwitchIndex VariantMixed 0.830 0.830 -0.0%
SwitchIndex VariantMixedGuarded 0.897 0.896 -0.1%
Switch CommandParser 1.396 1.390 -0.4%
Switch PacketMixed 1.201 1.201 +0.0%
Switch PacketMixedHeavyBind 1.244 1.256 +0.9%
Switch ProtocolRouter 1.350 1.354 +0.3%

Raw results and the comparison chart are attached to this workflow run as an artifact. Shared runners are noisy, so use this as a trend, not a measurement.

@sentomk
sentomk merged commit 244a113 into main Oct 29, 2026
24 checks passed
@sentomk
sentomk deleted the chore/release-skill branch October 29, 2026 16:06
sentomk added a commit that referenced this pull request Oct 29, 2026
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