docs: agent onboarding runbook + deaf-agent nudge warnings#49
Conversation
…nfigs Lessons from @grok's onboarding (2026-07-19), captured before the next agent (Kimi) arrives: - docs/AGENT-ONBOARDING.md: the full checklist - own identity mint (with the /agents stub trap and the right key-validity check), room join via members endpoint or invite code, per-agent poller config, the wake-path table per runtime (hook / tmux nudge / native scheduler / GUI command), the live no-poke test that defines done, and self-serve key rotation. - rooms watch now warns loudly when nudge_mode is 'none' (poller runs, files fill, agent never wakes - a human ends up typing 'check rooms' forever) and when nudge_mode 'command' has no command set. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0e5722f4c2
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| // The deaf-agent trap (bit @grok's onboarding, 2026-07-19): the poller runs | ||
| // and files fill up, but nothing ever wakes the agent, so it only responds | ||
| // when a human types into its CLI. Make that state impossible to miss. | ||
| if (nudgeMode === 'none') { |
There was a problem hiding this comment.
Put nudge warnings in the active poller
For the documented ide-agent-kit rooms watch path, these warnings never run because bin/cli.mjs imports startRoomPoller from ../src/team-relay/room-poller.mjs, and a repo-wide search shows no imports of src/room-poller.mjs. That means users with nudge_mode: "none" or an empty command config still get the old startup output and miss the new deaf-agent warning; the same check needs to be added to src/team-relay/room-poller.mjs (or the CLI needs to use this implementation).
Useful? React with 👍 / 👎.
Captures today's @grok onboarding lessons before Kimi arrives — see commit message for the item list. The runbook is the checklist petrus asked for in-room ("did you update iak repo with the latest from grok onboard? Kimi coming now!").
Review: cross-model pass, petrus merges. The checklist is also posted in thinkoff-development so Kimi's onboarding can use it before this merges.
🤖 Generated with Claude Code