Skip to content

v3.3.0.3 — grep fallback --type filter never matched - #64

Merged
savvides merged 1 commit into
mainfrom
release/v3.3.0.3
Aug 6, 2026
Merged

v3.3.0.3 — grep fallback --type filter never matched#64
savvides merged 1 commit into
mainfrom
release/v3.3.0.3

Conversation

@savvides

@savvides savvides commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Release commit for the last of the May/June bot queue.

What shipped

#41  first behavioral coverage for bin/idstack-learnings-delete
#37  first behavioral coverage for bin/idstack-learnings-promote
#42  --keyword / --cross-project coverage for idstack-learnings-search
#38  grep -- and tail -n hygiene in the fallback
#47  three untested branches of bin/idstack-timeline-log
#63  #43 redone on top of #38, plus the bug it exposed

The bug

Deduplicating the grep fallback meant writing the first test that ever ran it. It failed immediately:

writer (idstack-learnings-log, json.dumps):  "type": "technical"
fallback grep pattern:                       "type":"technical"

--type matched nothing idstack had written. Only reaches installs without python3, which is why it sat unnoticed — and why it's not promoted on the landing page.

Coverage

test/integration-test.sh went 25 → 48 assertions. The fallback path went from zero coverage to four assertions.

smoke              371/371
integration         48/48
setup               17/17
manifest-merge      23/23
version-classifier  27/27
plugin-status        9/9
preamble-python      6/6
mutation           14/14 guarded

Open queue is now 3, all needs-rework: #50, #54, #55.

🤖 Generated with Claude Code

Ships the last mergeable work from the May/June bot queue (#41, #37, #42,
#38, #47) plus #63, which redid #43 and fixed the bug that redo exposed:
the grep fallback filtered on "type":"technical" while idstack-learnings-log
writes "type": "technical" via json.dumps, so --type matched nothing on any
machine without python3.

Landing page carries the version but no new What's-new paragraph — the fix
only reaches installs without python3, too narrow to promote.

Verified: smoke 371/371, integration 48/48 (25 at v3.3.0.2), setup 17/17,
manifest-merge 23/23, version-classifier 27/27, plugin-status 9/9,
preamble-python 6/6, mutations 14/14 guarded.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
@savvides
savvides merged commit 6614d90 into main Aug 6, 2026
8 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