Skip to content

Show a lane with no live process as stopped on run:board, not running #3893 - #3898

Merged
joshuafolkken merged 1 commit into
mainfrom
3893-lane
Oct 15, 2026
Merged

joshuafolkken merged 1 commit into
mainfrom
3893-lane

Conversation

@joshuafolkken

Copy link
Copy Markdown
Owner

closes #3893

closes #3893

概要

  • run:board は、子のプロセスがないレーンの行を 🔄 running ではなく 🛑 stopped で表示する。🔧 の計時も止まる。判定は lane:prune / lane:await と同じプローブ(lane_spent.idle_lanes)で、応答のないプローブは running のまま
  • 閉じた(マージ済み)子は、レーンが残っていても merged / done で表示する
  • 空きレーン数は backlog_ready.free_lanes_beside に一本化し、lane:limit と同じ読みにした

実機証跡

プローブがプロセスなし(none)と読んだレーンの行。レーンの中から実行すると、macOS の pgrep は自分の祖先を除くため、このセッション(3893)は none と読まれる。その行が 🛑 になった:

pnpm josh run:board --chat

▶ backlogrun  ⏰ 1:18:56  ✅  6/10 ■■■■■■────  🔄 3  ⏳ 1  💤 0
  🔨 3875 ✨ Remove the causes of the top three guard refusa…  55:28  🧠3.3G    00:10  🔧 00:02
  💥 3867 ✨ Bound guard refusal and notice texts to the act…  49:58  🧠0.6G    00:32  🔧 00:06
  🛑 3893 🐛 Show a lane with no live process as stopped on …
  🔍 3865 ✨ Add a concise-prose rule to the principles        01:29  🧠0.2G    01:29  🔧 00:02

祖先の外(launchd に付け替えたプロセス)から実行すると、4 レーンとも子が見つかり running のまま。空き数は lane:limit と一致する:

pnpm josh lane:limit

lane limit 6 (JOSH_LANE_LIMIT) · in use 4 · free 2

pnpm josh run:board --chat

▶ backlogrun  ⏰ 1:19:42  ✅  6/10 ■■■■■■────  🔄 4  ⏳ 0  💤 0
  🚢 3875 ✨ Remove the causes of the top three guard refusa…  56:14  🧠1.3G    00:15  🚢 00:15
  🚢 3867 ✨ Bound guard refusal and notice texts to the act…  50:43  🧠0.3G    00:04  🔧 00:00
  🚢 3893 🐛 Show a lane with no live process as stopped on …  10:06  🧠1.9G    00:52  🔧 00:02
  🔍 3865 ✨ Add a concise-prose rule to the principles        02:14  🧠1.5G    02:14  🔧 00:02

テスト

  • run-board-status-idle.test.ts — プロセスのないレーンは stopped(回帰テスト)、閉じたものは merged
  • backlog-ready.test.ts — free_lanes_beside が lane:limit の表示する free と同じ値
  • lane-spent.test.ts / run-board-read.test.ts / run-board-reason.test.ts — idle_lanes、読みの配線、停止行の重なり保持

🤖 Generated with Claude Code

@joshuafolkken joshuafolkken added the bugfix Compatible bug fix label Oct 15, 2026
@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 21 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: joshuafolkken/kit/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 61370b36-6ab0-40d6-b43c-66d7afb5cfe1
📥 Commits

Reviewing files that changed from the base of the PR and between 5756102 and 8fefecd.

📒 Files selected for processing: 20
  • .josh/metrics-accepted/3893.json
  • .josh/observations/3871-comment-rounds.md
  • .josh/observations/3893.md
  • docs/josh-commands-run.md
  • scripts/backlog/backlog-ready.test.ts
  • scripts/backlog/backlog-ready.ts
  • scripts/lane/lane-spent.test.ts
  • scripts/lane/lane-spent.ts
  • scripts/run/board/run-board-github.ts
  • scripts/run/board/run-board-idle.test.ts
  • scripts/run/board/run-board-idle.ts
  • scripts/run/board/run-board-read.test.ts
  • scripts/run/board/run-board-read.ts
  • scripts/run/board/run-board-reason.test.ts
  • scripts/run/board/run-board-reason.ts
  • scripts/run/board/run-board-state.ts
  • scripts/run/board/run-board-status-idle.test.ts
  • scripts/run/board/run-board-status.ts
  • scripts/run/board/run-board-tick-idle.test.ts
  • scripts/run/board/run-board-tick.ts

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings: 1
Validation error: Unrecognized key: "tools"
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@joshuafolkken
joshuafolkken merged commit b6f49b2 into main Oct 15, 2026
22 checks passed
@joshuafolkken
joshuafolkken deleted the 3893-lane branch October 15, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Compatible bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show a lane with no live process as stopped on run:board, not running

1 participant