Skip to content

Session Handoff [hw-cli]: hw CLI merged with dev, reviewed and hardware-tested on the fork; upstream waits on Pieter's go; #101 awaiting owner #49

Closed

Description

@ptr727

Next steps, in priority order

  1. Monitor rtc: write the RX8130CE and YSN8900 weekday one-hot liquidraver/ZephCore#101 and answer the owner's feedback, unchanged from round 8 (Session Handoff [hw-cli]: liquidraver/ZephCore#101 awaiting owner review; #98 worktrees retired, shared Zephyr tree re-homed to r6-base #48): fix on work/rx8130ce-week-one-hot first, then re-squash the clean branch with a lease on Pieter's go. Before rebuilding in rx8130ce-week, delete its stale build folders (Session Handoff [hw-cli]: liquidraver/ZephCore#101 awaiting owner review; #98 worktrees retired, shared Zephyr tree re-homed to r6-base #48, What not to repeat). Done means rtc: write the RX8130CE and YSN8900 weekday one-hot liquidraver/ZephCore#101 is merged or the owner asks for nothing more.
  2. hw CLI: wait for Pieter's go before anything goes upstream. Pieter, 2026-10-05: "continue working on hw cli local branches and local pr only, not ready to file upstream yet before manual testing." The RAK4631 hardware pass is done (see What the last round did). When he says go:
    • cut a clean branch fresh from the then-current dev and squash work/hw-cli onto it, per the hub's upstream-contribution-workflow;
    • re-check line endings: git diff --shortstat must equal git diff -w --shortstat;
    • open the upstream PR only with his explicit say-so naming it.
      Done means a clean upstream PR exists, or Pieter asks for more testing first.
  3. If dev moves before then, merge it into work/hw-cli (Add hw hardware-report CLI command (iteration branch) #1) and hw-cli-only (Add the hw hardware-report CLI command (rebased, hw CLI only) #16) as this round did. Keep the two trees byte-identical by taking Add hw hardware-report CLI command (iteration branch) #1's resolution into Add the hw hardware-report CLI command (rebased, hw CLI only) #16 (git checkout <#1 head> -- <paths>). Run a recorded local pass, push, and request Copilot on both. CodeRabbit allows one per hour, and Add hw hardware-report CLI command (iteration branch) #1 goes first.
  4. When rtc: write the RX8130CE and YSN8900 weekday one-hot liquidraver/ZephCore#101 merges, close fork rtc: write the RX8130CE weekday one-hot #45 and RTC save writes the weekday in binary; the RX8130CE's WEEK register is one-hot #39, and delete the rtc: write the RX8130CE and YSN8900 weekday one-hot liquidraver/ZephCore#101 branches and worktrees on Pieter's go. Keep r6-base, which holds the shared Zephyr tree.
  5. Hold companion: mark a CLI reply cut to the app frame (iteration branch) #27-companion: never treat an over-the-air CLI command as local (iteration branch) #32 until Pieter says go.

External blockers

  • Pieter's go-ahead for any upstream PR (the hw CLI included), any force-push of a clean upstream branch, and any branch deletion.
  • liquidraver's review of rtc: write the RX8130CE and YSN8900 weekday one-hot liquidraver/ZephCore#101.
  • CodeRabbit on the fork: one review an hour per repository, and it must be requested (@coderabbitai review). Its clean pass shows only as an edit to its summary comment naming the head.

Internal dependencies

State (re-derive: git worktree list, gh pr list, pr_review.py status)

The parked decision queue

0 issues carry decision on this fork.

What the last round did

What not to repeat

  • Hub write scripts refuse to run from an exported copy. pr_review.py wait --request from a git archive export returned OUT_OF_SCOPE, because it derives the owner scope from its own checkout's origin. Run hub tooling from a fresh detached hub worktree at origin/main.
  • The round-1 fix introduced the round-2 findings. Staging only s_state left s_active published mid-probe. When a reviewer reports a shared-state race, publish every field a reader combines, together.

New learnings

  • None durable beyond this chain. Host facts (board fittings, the hub-worktree requirement) go to the maintainer's host memory.

Activity

  1. added
    handoffA link in the session handoff chain, one open issue per track
    on Oct 6, 2026
  2. ptr727 commented on Oct 10, 2026

    @ptr727
    OwnerAuthor

    Superseded by #58 (round 10). liquidraver#101 merged 2026-10-06; this round's state is there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    handoffA link in the session handoff chain, one open issue per track

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions