Skip to content

developer-experience: wire real merge state into ledger signals (zeroMergeStreak always-true bug) - #8

Closed
jjohare wants to merge 2 commits into
mainfrom
dream/2026-08-14-developer-experience
Closed

developer-experience: wire real merge state into ledger signals (zeroMergeStreak always-true bug)#8
jjohare wants to merge 2 commits into
mainfrom
dream/2026-08-14-developer-experience

Conversation

@jjohare

@jjohare jjohare commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Dreaming work preserved from the migration off ruvnet/dream-machine (was upstream PR #9). Recreated here as a draft so the in-flight dream cycle is not lost. Authored by the dreaming system (Claude).

🤖 Generated by Claude Code

claude added 2 commits August 14, 2026 09:14
…roMergeStreak always-true bug)

Nightly Dream Cycle, 2026-08-14. DEEP=developer-experience, SCAN=cli,tui.

`dream-machine ledger signals` never passed mergedPrNumbers into
learningSignals(), so zeroMergeStreak was a worst-case default (true
whenever the window has a real PR) presented as a verified reading.
Reproduced first-hand against this repo's real ledger: the CLI reported
zeroMergeStreak=true even though PR #7 is confirmed merged via the
GitHub API. Adds an optional `--merged "7,12"` flag on `ledger signals`
that threads known-merged PR numbers into the library's existing,
already-correct learningSignals(rows, { mergedPrNumbers }) option.

Full report: docs/dream-cycle/2026-08-14-developer-experience-report.md
@jjohare

jjohare commented Aug 16, 2026

Copy link
Copy Markdown
Contributor Author

Dreaming cycle integrated into main: code in d896bd3 (--merged wiring, Claude authorship preserved via cherry-pick), ledger row in e8088d9. Closing; work is on main.

@jjohare jjohare closed this Aug 16, 2026
@jjohare
jjohare deleted the dream/2026-08-14-developer-experience branch August 16, 2026 09:34
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.

2 participants