Skip to content

prep(v0.2.1): decision-matrix test coverage + changelog cleanup — Release Pending (2026-04-09)#12

Merged
saurabhjain1592 merged 2 commits intomainfrom
prep/review-fixes
Apr 8, 2026
Merged

prep(v0.2.1): decision-matrix test coverage + changelog cleanup — Release Pending (2026-04-09)#12
saurabhjain1592 merged 2 commits intomainfrom
prep/review-fixes

Conversation

@saurabhjain1592
Copy link
Copy Markdown
Member

Summary

v0.2.1 prep — Release Pending (2026-04-09). Not released here.

Review findings addressed

H3 — Bash plugin test coverage gap

  • Added decision-matrix mock-server cases for FAIL_CLOSED_METHOD (-32601), FAIL_CLOSED_PARAMS (-32602), FAIL_OPEN_INTERNAL (-32603), FAIL_OPEN_PARSE (-32700), FAIL_OPEN_UNKNOWN (-99999)
  • All 5 branches introduced in v0.2.0's fail-open hardening are now tested end-to-end

CHANGELOG internal refs

  • 'issue #1545 Direction 3' reference stripped from v0.2.0 entry
  • v0.2.0 date corrected 2026-04-09 → 2026-04-08

Manifest

  • .codex-plugin/plugin.json: 0.2.0 → 0.2.1

Tests

29/29 pass (up from 22). New cases cover the full decision matrix.

Not released

No tag, no publish, no merge.

v0.2.1 prep for the planned 2026-04-09 release. Not cut here.

Review finding H3:
- The v0.2.0 release added a new decision matrix for hook fail-open /
  fail-closed behavior but only updated a single stderr-string assertion
  in the test suite. All 5 new branches (curl timeout already tested,
  but -32603, -32700, -32601, -32602, unknown code) were untested.
- Added 5 new mock-server cases with FAIL_CLOSED_METHOD, FAIL_CLOSED_PARAMS,
  FAIL_OPEN_INTERNAL, FAIL_OPEN_PARSE, FAIL_OPEN_UNKNOWN trigger strings.
- Tests now assert the full decision matrix end-to-end.

Changelog cleanup (review finding, CLAUDE.md rule):
- Removed "issue #1545 Direction 3" internal ref from v0.2.0 section.
- v0.2.0 entry date corrected 2026-04-09 → 2026-04-08.

Manifest bump:
- .codex-plugin/plugin.json: 0.2.0 → 0.2.1

Tests: 29/29 pass (up from 22 in v0.2.0).
The v0.2.1 prep commit updated CHANGELOG and tests but the earlier
Edit on plugin.json failed silently (file-not-yet-read error) and
the file was left at 0.2.0. Fixing so the manifest version matches
the changelog entry.
@saurabhjain1592 saurabhjain1592 merged commit 25fcdc4 into main Apr 8, 2026
3 checks passed
@saurabhjain1592 saurabhjain1592 deleted the prep/review-fixes branch April 10, 2026 11:51
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