Skip to content

docs(analog): drop a duplicate lesson, add the pane-id and load lessons - #12336

Merged
gHashTag merged 1 commit into
masterfrom
lane/analog-skill-lessons-3
Oct 25, 2026
Merged

gHashTag merged 1 commit into
masterfrom
lane/analog-skill-lessons-3

Conversation

@gHashTag

Copy link
Copy Markdown
Collaborator

Summary

Follow-up to PR 12316 (merged 2026-10-25 07:27:43Z, all checks green). Its review summary comment pointed at two near-identical lessons in .claude/skills/analog-emulation/SKILL.md (the home-path rule for notes under docs/, refs 12116 and 10486). The comment was right; this PR removes the older, less precise line and keeps the one that names the Users directory.

Two lessons from loop ticks 54 and 55 are added (both measured in this loop, ref 10486):

  • T27OS_CTL=spawn prints "pane spawned pane N"; only the trailing number is the pane id. Using the whole string dropped a typed chain without an error and cost 6 minutes (13:59 to 14:05 +07).
  • If the load stays above 60 after the loop's own gate is killed, it is other sessions' load; a chain typed ahead dies with the pane, so the restart is queued for the next tick.

Checked

  • Diff: 1 line removed, 2 lines added, one file. Both new lines went in through the lesson command, which refuses duplicates and lines over 400 bytes.
  • No home path and no non-ASCII character in the diff.

Not claimed

  • The lessons are about this machine's loop (panes, load) and are not general rules; the load threshold is the loop's own kill rule.

Refs #10486. Part of epic #10486.

🤖 Generated with Claude Code

…ns (Refs #10486)

PR 12316 added two near-identical lessons about home paths in docs/ notes
(refs 12116 and 10486); the review summary comment on that PR flagged them.
The older, less precise one is removed. Two lessons from loop tick 54 and 55
are added through the lesson command: the pane id of a spawn is the trailing
number only, and a load that stays above 60 after the gate is killed belongs
to other sessions.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-10-25 07:58:32 UTC

Summary

Status Count
Total Open PRs 50
PRs with Failing Checks 22
PRs with All Checks Green 28
READY 16
FAILING 22
PENDING 0
NO CHECKS YET 0

These columns do not partition: 16 + 22 + 0 + 0 = 38, and there are 50 open PRs. A PR is being counted twice or not at all.

Seal Status

  • ⚠️ STALE -- sha256(compiler.rs)=55c0f1b1b32d != manifest seal=87e5cbd3ad94.
    The committed NMSE numbers were certified against an older compiler.rs.
    Run scripts/reseal-check.sh locally for the two-step reseal command (advisory; not a merge gate).

@tri-claw

tri-claw Bot commented Oct 25, 2026

Copy link
Copy Markdown

Summary by t27-bees

This PR removes a duplicate lesson about home-directory paths in docs/ (keeping the more precise version referencing the Users directory) and adds two new lessons from loop ticks 54 and 55 about pane ID extraction from T27OS_CTL=spawn output and load-based restart queuing.

Worth a look

  • .claude/skills/analog-emulation/SKILL.md:186 — The removed line references ref 12116 while the kept line references 10486; verify the kept lesson's context (CI scan refusing paths under Users directory) still matches current CI behavior.
  • .claude/skills/analog-emulation/SKILL.md:197 — The new lesson says "pass only the trailing number as T27OS_PANE"; confirm the code that parses T27OS_CTL=spawn output actually expects just the number and not the full string.
  • .claude/skills/analog-emulation/SKILL.md:198 — The load threshold of 60 for "other sessions' load" and restart queue threshold of 20 are specific to this machine's loop; if these values are hardcoded elsewhere, they should stay in sync.

Read: 1 files, +2 -1, the whole diff. Head 2eb1aeb. Model: openai-compatible/nvidia/nemotron-3-ultra-550b-a55b. Commands: @t27-bees review | summary | help | pause | resume. Free for every repository.

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