samagotchi 0.102.0 → 0.103.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (83) hide show
  1. checksums.yaml +4 -4
  2. data/CHANGELOG.md +40 -1
  3. data/bin/chi +23 -191
  4. data/docs/architecture.md +54 -0
  5. data/docs/cli.md +114 -7
  6. data/docs/configuration.md +7 -1
  7. data/docs/guardrails.md +2 -1
  8. data/docs/internals/engine.md +1 -1
  9. data/docs/internals/llm-context-forget.md +11 -1
  10. data/docs/internals/remote-sessions.md +303 -0
  11. data/docs/internals/working-set.md +95 -11
  12. data/docs/plugins.md +26 -2
  13. data/docs/sessions.md +12 -5
  14. data/docs/sub-agent.md +8 -0
  15. data/lib/samagotchi/answer_command.rb +27 -25
  16. data/lib/samagotchi/bridge.rb +24 -4
  17. data/lib/samagotchi/bridge_client/event_stream.rb +81 -12
  18. data/lib/samagotchi/bridge_client/stream_request.rb +77 -0
  19. data/lib/samagotchi/bridge_client.rb +22 -38
  20. data/lib/samagotchi/bundles/claude-peer/manifest.yml +2 -2
  21. data/lib/samagotchi/bundles/claude-peer/scripts/pair +30 -1
  22. data/lib/samagotchi/bundles/codex-peer/manifest.yml +2 -2
  23. data/lib/samagotchi/bundles/codex-peer/scripts/pair +49 -4
  24. data/lib/samagotchi/bundles/guardrails/guardrails/rules.yml +7 -4
  25. data/lib/samagotchi/bundles/guardrails/manifest.yml +1 -1
  26. data/lib/samagotchi/bundles/system/manifest.yml +1 -1
  27. data/lib/samagotchi/chi_invocation.rb +43 -0
  28. data/lib/samagotchi/cli/command.rb +3 -2
  29. data/lib/samagotchi/cli/flags.rb +5 -0
  30. data/lib/samagotchi/cli/parent_wait.rb +22 -13
  31. data/lib/samagotchi/context_command.rb +80 -73
  32. data/lib/samagotchi/engine.rb +12 -8
  33. data/lib/samagotchi/idle_recap.rb +7 -1
  34. data/lib/samagotchi/kernel_loop.rb +13 -1
  35. data/lib/samagotchi/launch_mode.rb +22 -0
  36. data/lib/samagotchi/llm_context_forget.rb +13 -6
  37. data/lib/samagotchi/note_command.rb +8 -7
  38. data/lib/samagotchi/parent_report.rb +55 -43
  39. data/lib/samagotchi/remote_command.rb +170 -0
  40. data/lib/samagotchi/remotes.rb +130 -0
  41. data/lib/samagotchi/run_options.rb +233 -0
  42. data/lib/samagotchi/send_command/busy.rb +58 -0
  43. data/lib/samagotchi/send_command/parent_gate.rb +23 -10
  44. data/lib/samagotchi/send_command/wait_target.rb +16 -3
  45. data/lib/samagotchi/send_command.rb +173 -108
  46. data/lib/samagotchi/session.rb +30 -5
  47. data/lib/samagotchi/session_access/contract.rb +129 -0
  48. data/lib/samagotchi/session_access/dispatch.rb +133 -0
  49. data/lib/samagotchi/session_access/http_transport.rb +109 -0
  50. data/lib/samagotchi/session_access/local.rb +349 -0
  51. data/lib/samagotchi/session_access/remote.rb +638 -0
  52. data/lib/samagotchi/session_access/web_bridge_client.rb +181 -0
  53. data/lib/samagotchi/session_access.rb +141 -0
  54. data/lib/samagotchi/session_archive_command.rb +9 -5
  55. data/lib/samagotchi/session_commands.rb +20 -15
  56. data/lib/samagotchi/session_delete_command.rb +8 -4
  57. data/lib/samagotchi/session_manager.rb +7 -2
  58. data/lib/samagotchi/session_setup.rb +11 -6
  59. data/lib/samagotchi/sessions_command.rb +36 -14
  60. data/lib/samagotchi/terminal_ui/attach_launcher.rb +51 -15
  61. data/lib/samagotchi/terminal_ui/attach_link.rb +104 -0
  62. data/lib/samagotchi/terminal_ui/attached_loop.rb +80 -75
  63. data/lib/samagotchi/terminal_ui/local_link.rb +130 -0
  64. data/lib/samagotchi/terminal_ui/version_lines.rb +5 -3
  65. data/lib/samagotchi/terminal_ui.rb +2 -2
  66. data/lib/samagotchi/title_command.rb +9 -11
  67. data/lib/samagotchi/version.rb +1 -1
  68. data/lib/samagotchi/web/app.rb +360 -69
  69. data/lib/samagotchi/web/public/data.js +3 -2
  70. data/lib/samagotchi/web/public/index.html +20 -4
  71. data/lib/samagotchi/web/public/stage_frame.js +7 -2
  72. data/lib/samagotchi/web/public/stage_view.js +3 -2
  73. data/lib/samagotchi/web/public/stream_handlers.js +3 -1
  74. data/lib/samagotchi/web/public/working_set.js +159 -28
  75. data/lib/samagotchi/web/public/working_set_rail.js +39 -12
  76. data/lib/samagotchi/web/public/working_set_view.js +91 -33
  77. data/lib/samagotchi/web/session_access_routes.rb +343 -0
  78. data/lib/samagotchi/worker_sidecar.rb +12 -5
  79. data/lib/samagotchi/working_set.rb +88 -14
  80. data/lib/samagotchi/working_set_command.rb +133 -30
  81. data/lib/samagotchi/working_set_pin.rb +34 -0
  82. data/lib/samagotchi/working_set_steer.rb +191 -27
  83. metadata +19 -1
data/docs/sessions.md CHANGED
@@ -31,7 +31,7 @@ A session is deleted if **expired by age OR overflow by count** (unless `keep_st
31
31
 
32
32
  **Stopping a worker:** `chi sessions stop ID` marks the session stopped (an empty `<id>/stopped` file, not a session.json field, which a worker saving at that moment would write over; a resume removes it), sends its worker TERM and waits (up to 10 s) until the worker has let go of `owner.lock`. A running turn is canceled over the Bridge first, with reason `session_stopped` (the turn's line ends "(session stopped)", its note says `cancelled (session-stopped)`, and a command it was running says "stopped as the session was stopped"; a bare signal to the worker makes it "stopped by a signal to chi"; an older worker takes the reason as `manual`); the TERM stops whatever still runs as a Stop with reason `signal` and saves it (a spawned worker traps TERM, HUP and INT: `WorkerSignals`; a turn not ended within 3 s, 0.5 s after a stop or a second signal, is cut and the session saved as it was), and a stopped worker still saves its kept turn, since the marker, not the saved status, says stopped. Any TERM, HUP or INT does the same to a worker outside a stop: it leaves (log `worker stop reason=signal`) and the next send wakes a new one, so a `chi --resume ID` right after spawns a fresh worker. That is also how to restart a worker still running an older chi, which the attached terminal reports when a command or a question dismiss gets a 404, and the web when an answer, a dismiss, a command or a cancel does (`501 not_supported`; `BridgeClient.stale_worker_message`). A command this chi has that the older worker doesn't (`/llm-context`, the web's `llm ctx` chip, against a 0.39.0 worker) says the same, not `not a session command` (`SessionCommands.builtin_name`); a typo keeps its `400 unknown_command`. The web's `POST /api/sessions/:id/stop` waits the same way, for 2 s. A command sent during a turn that waits for its end (`/model X`, `!cmd`; see *Typing during a turn* in [cli.md](cli.md)) and hasn't run when the worker leaves (a stop, a crash) is answered `command_ran {status: "dropped"}`, so no UI keeps showing it waiting.
33
33
 
34
- **Deleting a session:** `chi sessions delete ID...`, `/exit --delete` in a terminal and the Web UI's delete all go through `SessionManager.delete_session`: it resolves a unique prefix, removes `<id>.json` and the whole `<id>/` dir, and returns what it removed. A session a plain REPL has open (`owner.lock` kind `tui`) is always refused. One a worker runs is refused unless the caller asks to stop it: then it stops the worker as `chi sessions stop` does and deletes once `owner.lock` is free (`--force` on the CLI, 10 s; the web always, 2 s; `/exit --delete` after the worker agreed to exit, 10 s). A worker that outlives the wait leaves the session in place. The web's route is `DELETE /api/sessions/:id` (200 `{status: "deleted", session_id, stopped}`; 409 `owned_by_tui` or `still_stopping`; 404).
34
+ **Deleting a session:** `chi sessions delete ID...`, `/exit --delete` in a terminal and the Web UI's delete all go through `SessionManager.delete_session`: it resolves a unique prefix, removes `<id>.json` and the whole `<id>/` dir, and returns what it removed. A session a plain REPL has open (`owner.lock` kind `tui`) is always refused. One a worker runs is refused unless the caller asks to stop it: then it stops the worker as `chi sessions stop` does and deletes once `owner.lock` is free (`--force` on the CLI, 10 s; the web page always, 2 s; `/exit --delete` after the worker agreed to exit, 10 s). A worker that outlives the wait leaves the session in place. The web's route is `DELETE /api/sessions/:id?force=true` (the page sends `force`; without it a live worker's session is 409 `worker_running`, as the CLI without `--force`; 200 `{status: "deleted", session_id, stopped}`; 409 `owned_by_tui` or `still_stopping`; 404).
35
35
 
36
36
  <a id="archiving-a-session"></a>**Archiving a session:** `chi sessions archive ID...`, `/archive` in a terminal and the Web UI's `archive` go through `SessionManager.archive_session`: it writes `<id>/archived` (`{"archived_at": …}`; not a session.json field, which every save rewrites) for the session and its delegated children. `Session.list` leaves archived sessions out unless `include_archived:`, and everything built on it follows: `chi sessions list` (`--archived` shows them, `[archived]`), the summaries (`--live`, json/tsv, the desktop panel, the agent's `list_sessions`) and the retention prune, which neither deletes nor counts them against `max_count`. `children_of` (max_children, `delegate_result`) still sees them, and `--resume`/`--attach ID`, `chi send`/`chi note ID` reach them as before. Refused: a turn running in the session or in one of its children (naming the child; 409 `busy` on the web), a prompt queued in `input/` (it would run on the next resume) or a live worker's step-limit question still open, in it or a child (409 `busy` too), a plain REPL owning it (`owned_by_tui`), a `chi scratch` session (`scratch`). A live idle worker is stopped first, and the marker is written even if it is still shutting down. Input a human typed brings a session back (`{"unarchived_at": …}`): a prompt from the web, an attached or plain terminal or `chi send` (origin `web:*`, `tui:*`, `cli:send`, none), steering merged into a running turn, a `/continue` answer or a question answered (the step-limit question too); a delegate's follow-up, a plugin's turn, a reminder, the idle recap, a note and just opening it don't. A child's input brings only the child back. The retention sweep ages an unarchived session from when it was unarchived, so unarchiving an old session doesn't let the next sweep take it. `/archive` leaves the session first, as `/exit` does, then archives it: in an attached terminal the worker is asked to exit (one that stays up for another UI is stopped by the archive; a running turn, a queued prompt or an open step-limit question refuses it), and an empty session is discarded instead; `chi scratch` refuses it. Typed while a turn runs, `/archive` waits for the turn's end (an attached terminal for the commands queued behind it too) and then archives. `chi sessions unarchive ID...` and the web's `unarchive` bring back the whole family. Web routes: `POST /api/sessions/:id/archive` (200 `{status, session_id, archived, stopped, discarded}`, ids) and `POST /api/sessions/:id/unarchive` (200 `{status, session_id, unarchived}`). The session hub and `GET /api/sessions` keep archived sessions, `archived: true` in their summaries: the page leaves them out of the strip, its count and the list at render, and "include archived" by the all-sessions search shows them, dimmed with an `archived` badge. The info bar marks a session whose live worker runs an older chi than the newest installed (`worker_version` in its summary): `chi 0.18.1 · Restart`, which asks to confirm and calls `POST /api/sessions/:id/restart` (a refusal's reason shows in a toast); a worker from before restarts shows `chi sessions stop ID` instead. The info bar reads `continue → · archive · stop · delete` (`unarchive` on an archived one; no `continue →` on a link continued already, see *Session chains*); archiving the open session stays on it, with an Undo toast.
37
37
 
@@ -152,11 +152,18 @@ system message marked `kind: turn_note` (the UIs don't show it; they showed the
152
152
  the provider (cyber). …]` when the provider refused it.
153
153
 
154
154
  After you drop or undrop an item of the working set between turns (`/working-set drop KEY`, the web list's
155
- `drop`; [cli.md](cli.md#the-working-set)), the next turn starts with one line about it (`kind: working_set`,
155
+ `drop`; [cli.md](cli.md#the-working-set)), or pin or unpin one under a strategy with the forget layer, the next turn
156
+ starts with one line about it (`kind: working_set`,
156
157
  escaped like a context note; the UIs don't show it): `[WORKING SET: since your last turn the user dropped
157
158
  lib/cart.rb from your context: you are sent a stub instead of its outputs; read it again only if the task needs
158
159
  it.]`. Every op since the last turn joins that one line, and an undrop before the next turn takes its drop off it.
159
- The drop itself is saved on the entries (`edits`, kind `drop`), so `--resume`, rollbacks and forks keep it.
160
+ An add (`/working-set add KEY`) asks the model in that line to load the item first (`the user added lib/tax.rb to
161
+ your context: before anything else, load it with read "/abs/lib/tax.rb"`); an unadd before the next turn takes it
162
+ off.
163
+ The drop itself is saved on the entries (`edits`, kind `drop`), so `--resume`, rollbacks and forks keep it. A pin is
164
+ the session file's `working_set_pins` (`[{"kind": "file", "key": "/abs/path"}]`, `[]` for none; kinds `file`,
165
+ `memory`, `url`), not an edit: a respawned worker reads it, a continue and a fork (`ctx.sessions.fork`) copy it as
166
+ they copy the toolkit, and a delegate child starts without. A rollback leaves it (it is not in the conversation).
160
167
 
161
168
  When the context window fills past 40% (then 60%, 80%), the native loop leaves a similar line (`kind: context`):
162
169
  `[CONTEXT: about 55% of the context window is in use (estimated; bucket=40plus). context moderate — prefer targeted
@@ -187,7 +194,7 @@ pbpaste | chi send 3fa2 # the clipboard is the messa
187
194
 
188
195
  - With both stdin and `-m`, stdin is context: each line becomes a `> ` quote (as web annotations quote a selection), then a blank line, then the message. With only one, it goes in as is. A terminal on stdin is ignored. Both empty is a usage error; over 16 KiB is refused, never cut.
189
196
  - It goes through the same path as the web composer (`SessionManager.deliver_turn`): the worker's Bridge when it is up, so every attached UI shows it as your message (client id `cli:send`, shown like any user message); the input file when the worker is on its way out. During a running turn it is merged into that turn at the next step, like a message typed then, and cuts nothing. `--cut` asks for a cut, as `/cut TEXT` in a terminal does: a generation that has streamed only thinking for `steer.cut_after` seconds (per host/model: `steer_cut_after`) is cut (now, or once its thinking passes that), and the step starts again with the message. To an idle session `--cut` sends normally. `--cut --wait` into a running turn is refused like any `--wait` there (`busy: a turn is running`). `--queue` (not with `--cut`, a usage error) keeps the message for after the running turn: it runs as a turn of its own then (as `/queue TEXT` does), and a plain message sent after it still goes in at the running turn's next step. To an idle session it runs at once (it still goes as a queue, so a turn another client starts first never takes it as a steer). A worker older than queueing would steer it instead, so it is refused (to an idle session too) (`refused: this session's worker predates queueing, …; restart it`, exit 1) and nothing is sent; `chi sessions restart ID` and send again.
190
- - A session with no worker gets one started (like `--attach` or the web composer). A session open in a plain REPL (`--no-shared`) refuses it. Only sessions on this machine: Bridges listen on 127.0.0.1.
197
+ - A session with no worker gets one started (like `--attach` or the web composer). A session open in a plain REPL (`--no-shared`) refuses it. Bridges listen on 127.0.0.1: another machine's sessions are reached through its chi web, `chi --remote NAME send …` ([Remote sessions](cli.md#remote-sessions)).
191
198
  - Fire and forget: it returns once the message is queued and never prints the answer; that shows in whatever is attached. A guardrail "ask" waits for a UI to answer it, so with nothing attached the turn stalls there until one attaches (`chi --attach ID`).
192
199
  - `--image PATH` (repeatable, up to 20) sends images with the message, as the web composer's chips do. Each file is read once before anything is sent (bmp, tiff and heic converted to png, large ones downscaled, as for `@path`); a missing file or one that isn't an image stops the send (exit 1; with `--wait --format json`, an `error` line) and nothing is sent; more than 20 is a usage error (exit 2). Each session gets its own copy in `<session>/images/`. Text is still required (`-m` or stdin; context alone counts), also with `--wait`. The session's model must see images: one known not to (the host's model list says text-only, `vision: false` in `models:` or `hosts:`, a native llama.cpp without a vision model) is refused before anything is sent, with the reason (`<id> refused: gemma can't take images (…); send text only or switch the model (/model)`); the other sessions of the same send still get it, and the exit is 1. With `--new` nothing is started. When chi can't tell (a native host whose `/props` doesn't answer, a list that doesn't say), the message is sent as before and a text-only model fails the turn in the session. A running turn doesn't take images mid-turn: the message runs as the next turn, and the line says `(runs after the current turn)`. With `--new` the session starts idle with the message as its preview, then the message goes in as its first turn once its worker is up (`<id> started with 1 image`); if the worker doesn't come up in 5 s the session is kept, with its id on the `failed:` line.
193
200
  - A session command runs as the command, as typed in a terminal: `chi send -m "/model qwen" 3fa2` switches the model (`sent as a session command`), and the model never sees the line. What counts is what the session's worker knows (the built-ins, `!cmd`, its bundles' commands); an unknown `/word` (`/usr/bin/env is missing`) goes in as a message. A UI-only command (`/stats`, `/exit`, `/recap`, …) is refused instead: the worker doesn't run it, so the line would reach the model as a prompt (`<id> refused: /stats is a terminal and web command; use chi sessions stats <id8>`; exit 1, and nothing is sent). With `--new` a line starting with `/` or `!` starts the session idle and goes to its worker once it is up, which decides. With `--wait` a command has no reply to wait for: exit 0 at once, `<id> sent as a session command: no reply to wait for …` on stderr (`{"status":"command",…,"detail":…}` with `--format json`); its output shows where the session is open.
@@ -243,7 +250,7 @@ A session can **continue** an earlier one: "the next run of the same routine" (a
243
250
  - **Its open delegates move to the new link** instead: each direct delegate that is running, waiting for an answer, has a live worker or a reply its parent wasn't given, or has such a delegate below it (checked again after the recap wait, so one delegated meanwhile moves too). It moves with its own delegates, stays unarchived, and from then on is the new link's: its card folds under the new link, its reports ring the new link (an idle one runs a wake turn for them, as for its own delegates), `delegate_result` and the `⑂` chip count it there, and what its parent was already told (`delegates.json`) and any report still waiting come along, so nothing is reported twice. A moved delegate gets a context note from `session chain`, `Your parent session is now 4b7e9a01 (…), the next link of its session chain: …`, as its system prompt keeps naming the parent it started with. The move writes `<child>/parent.json` (`{parent_id, from, at}`) beside the child's session file, never the file itself, so a live child worker saving its own copy changes nothing; every reader applies it (a later move overwrites it; it is never removed). A continue is **refused for an open delegate that can't move** (`SessionContinue::Refused`, naming it): one whose live worker runs an older chi (`older chi worker 0.46.1; restart it`: `chi sessions restart <id>`, or stop it) or that is open in a chi REPL (close it there). An older `chi web` or CLI still running shows a moved delegate under the archived link until it is restarted.
244
251
  - **If the continue fails part-way:** a new link that fails to start gives its moved delegates back before it is deleted (each gets a note saying so; its worker is stopped first, if one started, and a worker that won't stop is reported, the link left), with any report one of them rang it with meanwhile; the previous link, back in the lists, is woken for reports waiting. A continue that dies in the middle leaves a `move.json` in the new link's folder: the new link's next worker finishes that move (only the delegates it names, and without a second note to one already told) as it starts. Until the new link's own worker holds it, a ring from a moved delegate doesn't start another (a `starting` marker in its folder, which that worker removes; a stale one, over 60 s old or from a process that is gone, is ignored). An open delegate below an archived delegate refuses the continue (`under archived …`): unarchive that one, or archive this one.
245
252
  - **Refused too:** a link that is continued already (naming the link that continues it; continue that one, or `last:<id>`), a previous link whose folder is gone, and one whose model's host can't run it when no default model is configured to fall back to. A refusal starts nothing and archives nothing; a new link that fails to start puts back what was archived. Every refusal that can be known up front (the archive's ones included) comes before the recap wait. One continue of a session runs at a time (a lock in its folder; other chains don't wait), so a double click finds the first one's link and is refused, while a `last:<id>` continue that waited follows the chain to its new end.
246
- - **Web:** `POST /api/sessions` takes `continues: "<id, prefix or last:id>"`, with `idle: true` (a new link waiting for its first message, previewed by the previous link's, or `preview`) or a `prompt`, and `model` (or `model_name`) for the new link's model, but not `dir`, `llm_context` or `thinking` (400 `invalid_continues`: the previous link's are taken). 201 is the new session's summary, `continues` set, with `moved`: the ids of the delegates that moved to it (`[]` for none), and `model_warning`: the default-model fallback's warning, then any about the model id, joined by `; ` (`null` for none), and `opener`: the previous link's first typed message, in full, when that link continues one itself (`null` otherwise: a chain's first link's first message is its task); 404 `not_found`; 400 `ambiguous_id` for a prefix of several sessions, `invalid_model` for a given model whose host config.yml doesn't have (or a fallback with no default model); 409 `continued` with `next_id` (the link that continues it already, which the page opens instead), `open_children` with `ids` (open delegates that can't move), `folder_gone`, `busy`, `scratch` or `owned_by_tui`. Every session summary carries `continues`.
253
+ - **Web:** `POST /api/sessions` takes `continues: "<id, prefix or last:id>"`, with `idle: true` (a new link waiting for its first message, previewed by the previous link's, or `preview`) or a `prompt`, and `model` (or `model_name`) for the new link's model, but not `dir`, `llm_context`, `thinking`, `settings`, `toolkit`, `tools`, `no_tools` or `parent_answers` (400 `invalid_continues`: the previous link's are taken). 201 is the new session's summary, `continues` set, with `moved`: the ids of the delegates that moved to it (`[]` for none), and `model_warning`: the default-model fallback's warning, then any about the model id, joined by `; ` (`null` for none), and `opener`: the previous link's first typed message, in full, when that link continues one itself (`null` otherwise: a chain's first link's first message is its task); 404 `not_found`; 400 `ambiguous_id` for a prefix of several sessions, `invalid_model` for a given model whose host config.yml doesn't have (or a fallback with no default model); 409 `continued` with `next_id` (the link that continues it already, which the page opens instead), `open_children` with `ids` (open delegates that can't move), `folder_gone`, `busy`, `scratch` or `owned_by_tui`. Every session summary carries `continues`.
247
254
  - **The page:** the info bar's `continue →` continues the open session (`continues: <id>, idle: true`; it reads `continuing…` through the recap wait) and opens the new link with an opener in the composer, not sent (a draft written in this session stays instead; one carried over from another session or the start page gives way to the opener, and the toast's **Restore draft** brings it back): the previous link's own first typed message when that link continues one (the 201's `opener`; a `!shell` line isn't taken), else `Continue where we left off.`. So whatever you sent on the last link comes back on the next one, and editing it once changes it for every later link; the previous link is archived, a toast says so (`Continued 3fa2c1d0 in 4b7e9a01; 3fa2c1d0 is archived; 1 open delegate moved`), and a moved delegate's card folds under the new link. When the previous link's model's host is gone the link starts on the default model and the toast carries the server's `model_warning` too, and then stays until closed. A fresh link (one with no turn yet, however it was made: `continue →`, `chi send --new --continues`, a reload) shows the model picker in its composer with the link's model, until its first turn starts (a failed or canceled first turn ends it too); a pick runs `/model <name>` in the link (a command bubble; the info bar follows) and changes neither the start page's remembered model nor its Recent list, and a Send right after it waits for the `/model` to be taken. A link a chi REPL owns, or one continued already, shows none. A link continued already has no `continue →`: its `→` link goes to the next link (a stale page's continue answered 409 `continued` opens `next_id`). A continue refused for open delegates that can't move shows a toast that names each one (its preview and what the server says to do: `older chi worker 0.46.1; restart it`, `open in a chi REPL; close it there`), each a link to its page; any other refusal shows the server's words. The info bar of a link carries `← 2026-10-08` (the previous link's day) and, on an earlier link, `2026-10-10 →`, each a link with its id, its full start (`2026-10-10 14:32`) and its preview in the tooltip; a neighbour that started on the open link's own day shows its time instead (`← 09:12`, `14:32 →`). A link made by `continue →` inherits the previous link's preview (and its title, if it has one) when it is created, so its card keeps its chain's title whatever opener it is sent, and its top bar and browser tab show its card's title. A link whose title is the opener and nothing else (a `chi send --new --continues` link whose first message is `Continue where we left off.`) shows its chain's first link's title instead, on its card, in the popover and in its info bar's title line, display only (its messages and data keep the opener; its own title when the first link is gone). The lists fold a chain into the card of its latest listed link, whatever the earlier links' archived state (typing in an archived link brings it back, folded there still), their delegates with them: the card's `↩ #N` chip (`#N of M` when later links aren't listed; its tooltip `Session chain: link N of M`) opens a popover of the other links, newest first (`#N`, its date, with its time (`2026-10-17 14:32`) when another link of the chain started the same day, its own title when it differs from the card's (a link inherits its chain's, so mostly it doesn't), else its recap's first sentence, else its preview, `archived`), a click opens one; the chip shows a live dot or `waiting` for a link that has a worker or waits on the user, and a waiting link puts its chain's card first. All sessions folds nothing while a search is typed or "include archived" is on: every link is a card of its own then, its chip `#N of M`, with its own delegates folded in, so a search finds an earlier link and its delegates.
248
255
  - **CLI:** `chi send --new --continues (ID|PREFIX|last:ID) [--model M] [-m TEXT]` (see *Starting a session*).
249
256
  - **Which link:** an id, a unique prefix, or `last:<id or prefix>`, the latest link of the chain that session is in. It walks the links that still exist: a deleted link in the middle (`chi sessions delete`, the retention sweep of an unarchived one) ends the walk before it, so `last:` of an earlier link then names that earlier link, and continuing it starts a second line beside the links after the gap; name a link after the gap instead. A trigger names the chain this way: `chi send` to an archived link would bring it back (a human's input), not continue the chain.
data/docs/sub-agent.md CHANGED
@@ -164,6 +164,14 @@ An answer typed into `chi --attach` or the REPL counts as a parent's too when
164
164
  stdin isn't a terminal or an agent marker (`CLAUDECODE`, `AI_AGENT`,
165
165
  `CODEX_THREAD_ID`, `SAMAGOTCHI_PARENT_SESSION`) is set.
166
166
 
167
+ Through `chi web` (`POST /api/sessions/:id/answer`, what chi answer uses against
168
+ another machine's sessions) the answer keeps chi answer's mark: the route passes
169
+ on `client_id: "cli:answer"` and drops any other id, so a forwarded answer can
170
+ only be held to the parent rules, never pass for the user's own. The worker's
171
+ refusal (403 `parent_answer_refused`) comes back as it said it. A parent's
172
+ `/continue yes` and a loosening `/parent-answers` sent through `chi web`'s create
173
+ and turn routes are refused there too, held to that machine's config.yml.
174
+
167
175
  The worker checks it again with its own config.yml. The guardrails settings
168
176
  have no environment variables, and a worker drops any `SAMAGOTCHI_GUARDRAILS_*`
169
177
  it inherits; but a parent that sets `XDG_CONFIG_HOME` picks which config.yml a
@@ -10,6 +10,7 @@ require_relative "guardrails/parent_policy"
10
10
  require_relative "cli/command"
11
11
  require_relative "cli/flags"
12
12
  require_relative "cli/parent_wait"
13
+ require_relative "session_access/local"
13
14
 
14
15
  module Samagotchi
15
16
  # `chi answer`: answer the question a session's worker waits on (exit 3
@@ -68,11 +69,14 @@ module Samagotchi
68
69
  f.value "--format"
69
70
  end
70
71
 
71
- def initialize(argv, stdout: $stdout, stderr: $stderr, state_dir: nil)
72
+ # @param access [SessionAccess::Local, nil] where the sessions are;
73
+ # default this machine's, in +state_dir+
74
+ def initialize(argv, stdout: $stdout, stderr: $stderr, state_dir: nil, access: nil)
72
75
  @argv = argv.dup
73
76
  @stdout = stdout
74
77
  @stderr = stderr
75
78
  @state_dir = state_dir || Session.default_state_dir
79
+ @access = access || SessionAccess::Local.new(state_dir: @state_dir)
76
80
  end
77
81
 
78
82
  private
@@ -106,23 +110,23 @@ module Samagotchi
106
110
  def run_parsed(options)
107
111
  id = resolve(options[:id]) or return 1
108
112
  qid = options[:question].strip
109
- session = Session.load(id, state_dir: @state_dir)
113
+ session = @access.show(id)
110
114
  pending = session.pending_question
111
115
  # Taken before the answer goes in, with the question answered as the
112
116
  # one already reported: the file keeps it until the turn thread
113
- # wakes, and it must not come back as exit 3.
114
- cursor = ReplyWait.newest_reply(id, state_dir: @state_dir)
115
- baseline = ReplyWait.baseline_of(session, question_id: qid)
116
- # The turn the answer is about: the one asking (running), or the one
117
+ # wakes, and it must not come back as exit 3. Its own_turn is the
118
+ # turn the answer is about: the one asking (running), or the one
117
119
  # whose continue offer stands. Its own reply is the answer's, not a
118
120
  # turn queued behind it; nil (an older worker): the next reply.
119
- turn = ReplyWait.own_turn(session, session_dir: Session.session_dir(id, state_dir: @state_dir))
121
+ start = @access.prepare_wait(id, session: session, question_id: qid)
120
122
 
121
123
  if pending && pending.id.to_s == qid
122
124
  return failure(not_dismissable(pending)) if options[:dismiss] && ParentReport.continue?(pending)
123
125
 
124
126
  selected = selection(pending, options) or return @selection_status
125
- refusal = parent_refusal(pending, selected, session)
127
+ # Remotely the worker's check alone (chi web passes chi answer's
128
+ # client_id on, and its 403 says why).
129
+ refusal = parent_refusal(pending, selected, session) unless @access.remote?
126
130
  return failure(refusal) if refusal
127
131
 
128
132
  posted = post(id, qid, selected, options)
@@ -134,7 +138,7 @@ module Samagotchi
134
138
  end
135
139
  # Answered here, or no longer open (answered elsewhere, or a newer
136
140
  # question): what comes next either way.
137
- wait_for_reply(id, cursor: cursor, baseline: baseline, timeout: options[:timeout], turn: turn)
141
+ wait_for_reply(id, cursor: start.cursor, baseline: start.baseline, timeout: options[:timeout], turn: start.own_turn)
138
142
  end
139
143
 
140
144
  # The labels to send. A number is an option's place (1-based); an
@@ -163,6 +167,8 @@ module Samagotchi
163
167
  # with this process's config.yml tightened by the session file's own
164
168
  # parent_answers, and the question's own tags). A deny, text alone or a
165
169
  # dismissal always passes. The worker checks it again with its own.
170
+ # Machine-local (this machine's config.yml): remotely skipped here; the
171
+ # server forwards CLI_ANSWER and the worker's check refuses (P2).
166
172
  def parent_refusal(pending, selected, session)
167
173
  offered = Array(pending.options).map(&:to_s)
168
174
  indices = selected.map { |label| offered.index(label) }
@@ -177,23 +183,18 @@ module Samagotchi
177
183
  "(--option Stop --text WHY stops it)"
178
184
  end
179
185
 
180
- # Post the answer (or the dismissal) to the live worker's Bridge.
186
+ # Post the answer (or the dismissal) to the live worker (its Bridge,
187
+ # through SessionAccess), marked as chi answer's.
181
188
  # @return [Integer, nil] an exit status when it didn't go in; nil to
182
189
  # wait (it went in, or the question was no longer open: 409)
183
190
  def post(id, qid, selected, options)
184
- client = BridgeClient.discover(id, session_dir: Session.session_dir(id, state_dir: @state_dir))
185
- return failure(worker_gone(id)) unless client
186
-
187
- reply = nil
188
- 2.times do
189
- reply = if options[:dismiss]
190
- client.dismiss_question(id: qid)
191
- else
192
- client.answer(id: qid, selected: selected, freeform: options[:text],
193
- client_id: ClientId::CLI_ANSWER)
194
- end
195
- break unless reply.status == 408
196
- end
191
+ reply = if options[:dismiss]
192
+ @access.dismiss(id, question_id: qid)
193
+ else
194
+ @access.answer(id, question_id: qid, selected: selected, freeform: options[:text],
195
+ client_id: ClientId::CLI_ANSWER)
196
+ end
197
+ return failure(worker_gone(id)) unless reply
197
198
 
198
199
  case reply.status
199
200
  when 200 then nil
@@ -209,7 +210,7 @@ module Samagotchi
209
210
  return failure(worker_gone(id)) if reply.json&.dig("error") == "unknown_session"
210
211
 
211
212
  cant = options[:dismiss] ? "dismiss a question" : "take an answer"
212
- failure(BridgeClient.stale_worker_message(id, cant: cant))
213
+ failure(BridgeClient.stale_worker_message(id, cant: cant, via: @access.invocation))
213
214
  else failure("the worker answered #{reply.status}: #{reply.json&.dig("detail") || reply.body}")
214
215
  end
215
216
  rescue Errno::ETIMEDOUT
@@ -229,7 +230,8 @@ module Samagotchi
229
230
  end
230
231
 
231
232
  def worker_gone(id)
232
- "the worker is gone and the question with it; send the task again: chi send --wait -m \"…\" #{id}"
233
+ "the worker is gone and the question with it; send the task again: " \
234
+ "#{@access.invocation.command("send --wait -m \"…\" #{id}")}"
233
235
  end
234
236
 
235
237
  # @return [Integer] 1, after the line
@@ -172,6 +172,9 @@ module Samagotchi
172
172
  # One a worker that died left is not open.
173
173
  @pending_card.clear
174
174
  @pending_card_handle = @engine.subscribe(observer: @pending_card)
175
+ # #stop and #close_intake wake the acceptor through it: closing the
176
+ # listen socket doesn't cut a select on it short on Linux.
177
+ @wake_reader, @wake_writer = IO.pipe
175
178
  @accept_thread = Thread.new { accept_loop }
176
179
  @accept_thread.report_on_exception = false
177
180
  write_sidecar
@@ -198,12 +201,17 @@ module Samagotchi
198
201
  rescue StandardError
199
202
  nil
200
203
  end
204
+ wake_acceptor
201
205
  end
202
206
 
203
207
  # Stop the acceptor, release the socket and remove the sidecar, so clients
204
208
  # don't have to find it stale. Does not stop the owning turn. Joins the
205
209
  # acceptor thread so the process can exit cleanly (Ruby waits for a thread
206
210
  # blocked in IO.select at VM shutdown). Safe to call again.
211
+ #
212
+ # The sidecar goes before the open streams close: a client that
213
+ # reconnects as its stream ends (chi web's proxy, an attached TUI) then
214
+ # finds no worker, or the next one, never this one's dead port.
207
215
  def stop
208
216
  @mutex.synchronize { @stopped = true }
209
217
  begin
@@ -211,6 +219,7 @@ module Samagotchi
211
219
  rescue StandardError
212
220
  nil
213
221
  end
222
+ wake_acceptor
214
223
  @capture_handle&.unsubscribe
215
224
  @accumulator_handle&.unsubscribe
216
225
  @engine.running_turn_messages = nil
@@ -220,13 +229,14 @@ module Samagotchi
220
229
  # A turn post killed between its enqueue and its reply looks failed to
221
230
  # the web, which then queues the prompt again from the input file.
222
231
  await_answers(REQUEST_GRACE_SECONDS)
232
+ remove_sidecar
223
233
  @connection_threads.each { |t| t.kill rescue nil }
224
234
  @connection_threads.clear
225
235
  # Its relay watchers (RelayWatcher) end with it.
226
236
  watchers = @mutex.synchronize { (@relay_watchers || []).tap { @relay_watchers = [] } }
227
237
  watchers.each { |thread| thread.kill.join(RELAY_WATCHER_JOIN_SECONDS) }
228
238
  @accept_thread&.join(2)
229
- remove_sidecar
239
+ [@wake_reader, @wake_writer].each { |io| io&.close rescue nil }
230
240
  nil
231
241
  end
232
242
 
@@ -360,17 +370,25 @@ module Samagotchi
360
370
  end
361
371
  end
362
372
 
373
+ # #stop or #close_intake: the acceptor's select returns now.
374
+ def wake_acceptor
375
+ @wake_writer&.write_nonblock(".", exception: false)
376
+ rescue IOError
377
+ nil # closed by an earlier #stop
378
+ end
379
+
363
380
  def accept_loop
364
381
  loop do
365
382
  break if stopped? || @server.closed?
366
383
 
367
384
  begin
368
- rs, = IO.select([@server], nil, nil, 0.5)
385
+ rs, = IO.select([@server, @wake_reader], nil, nil, 0.5)
369
386
  rescue IOError
370
387
  break
371
388
  end
372
389
 
373
390
  next unless rs
391
+ break if rs.include?(@wake_reader)
374
392
 
375
393
  client =
376
394
  begin
@@ -1264,9 +1282,11 @@ module Samagotchi
1264
1282
 
1265
1283
  def write_sidecar
1266
1284
  # version: the chi this worker runs (chi update and the web's badge
1267
- # report older ones); features: what a client may ask of it.
1285
+ # report older ones); features: what a client may ask of it; epoch:
1286
+ # this worker's, as its event ids carry it.
1268
1287
  WorkerSidecar.new(port: @port, bind: @bind, session_id: @session_id, started_at: Time.now.iso8601(3),
1269
- version: Samagotchi::VERSION, input_format: @input_format, features: FEATURES).write(session_dir)
1288
+ version: Samagotchi::VERSION, input_format: @input_format, features: FEATURES,
1289
+ epoch: @epoch).write(session_dir)
1270
1290
  rescue StandardError => e
1271
1291
  Log.warn(:bridge, "sidecar_write_failed", echo: "Bridge: failed to write #{WorkerSidecar::FILE}: #{e.class}: #{e.message}", error: e.class.name)
1272
1292
  end
@@ -19,11 +19,22 @@ module Samagotchi
19
19
  # it without that event.
20
20
  #
21
21
  # With +rediscover+, a failed connection first looks for the session's
22
- # live worker (its sidecar): a new one (a restart) is followed from its
23
- # snapshot, announced by a synthetic `{"type" => "worker_changed",
24
- # "port" => ...}` first, since its event ids start over. (A new worker
25
- # on the old one's port is the plain reconnect: its Bridge can't replay
26
- # the old ids and sends a reset.)
22
+ # live worker (its sidecar): a new one (a restart: a client with another
23
+ # BridgeClient#worker_key) is followed from its snapshot, announced by a
24
+ # synthetic `{"type" => "worker_changed", "port" => ...}` first, since its
25
+ # event ids start over. (A new worker on the old one's port is the plain
26
+ # reconnect: its Bridge can't replay the old ids and sends a reset.)
27
+ # Rediscovery may also give a reason ([nil, reason] while it retries,
28
+ # [:gone, reason] to stop): the stream_closed event carries the last one.
29
+ #
30
+ # Off by default, for a stream through chi web (SessionAccess::
31
+ # WebBridgeClient), where the client stays the same object across
32
+ # workers: +epoch_watch+ (a frame whose id has another epoch than the
33
+ # cursor's is a new worker: worker_changed first, the client's key
34
+ # updated), +announce_retries+ (stream_retrying once a reconnect fails,
35
+ # stream_resumed at the next frame), +idle_timeout+ (no byte, not even
36
+ # the Bridge's 15 s ping, for that long: a half-open connection,
37
+ # reconnected with the cursor).
27
38
  class EventStream
28
39
  DEFAULT_RECONNECT_DELAYS = [0.1, 0.25, 0.5, 1.0, 2.0].freeze
29
40
  # Long enough for a restarted worker to boot (its Engine, the plugins).
@@ -40,12 +51,26 @@ module Samagotchi
40
51
  # @param reconnect_delays [Array<Float>] sleeps between failed attempts;
41
52
  # one more failure than there are delays gives up
42
53
  # @param rediscover [#call, nil] → a BridgeClient for the session's live
43
- # worker now, nil when there is none yet, or :gone to give up now
54
+ # worker now, nil when there is none yet, or :gone to give up now;
55
+ # or AttachLink#live_worker's pair, [one of those, a reason]
56
+ # @param epoch_watch [Boolean] see the class comment; the client takes
57
+ # worker_key= then
58
+ # @param announce_retries [Boolean] see the class comment
59
+ # @param idle_timeout [Float, nil] seconds; nil: a quiet stream is
60
+ # never cut (a Bridge on this machine)
44
61
  # @yieldparam event [Hash] string-keyed event
45
62
  def initialize(client, snapshot: true, client_id: nil, reconnect_delays: DEFAULT_RECONNECT_DELAYS, rediscover: nil,
46
- &on_event)
63
+ epoch_watch: false, announce_retries: false, idle_timeout: nil, &on_event)
47
64
  @client = client
65
+ # The worker followed: a client that keeps its object across
66
+ # workers (epoch_watch) changes its key, not itself.
67
+ @worker_key = client.worker_key
48
68
  @rediscover = rediscover
69
+ @epoch_watch = epoch_watch
70
+ @announce_retries = announce_retries
71
+ @idle_timeout = idle_timeout
72
+ @retrying = false
73
+ @reason = nil
49
74
  params = []
50
75
  params << %w[snapshot 1] if snapshot
51
76
  params << ["client_id", client_id] if client_id
@@ -96,18 +121,21 @@ module Samagotchi
96
121
 
97
122
  if outcome == :events
98
123
  failures = 0
124
+ @reason = nil
99
125
  next
100
126
  end
101
127
 
128
+ announce_retry
102
129
  case rediscover_worker
103
130
  when :moved
104
131
  failures = 0
132
+ @reason = nil
105
133
  next
106
- when :gone then return finish("unreachable")
134
+ when :gone then return finish(@reason || "unreachable")
107
135
  end
108
136
 
109
137
  failures += 1
110
- return finish("unreachable") if failures > @reconnect_delays.size
138
+ return finish(@reason || "unreachable") if failures > @reconnect_delays.size
111
139
 
112
140
  sleep(@reconnect_delays[failures - 1])
113
141
  end
@@ -128,11 +156,15 @@ module Samagotchi
128
156
  yielded = false
129
157
  parser = SSEParser.new
130
158
  loop do
159
+ raise Errno::ETIMEDOUT if @idle_timeout && !sock.wait_readable(@idle_timeout)
160
+
131
161
  chunk = sock.readpartial(4096)
132
162
  parser.feed(chunk) do |frame|
133
163
  event = parse(frame[:data])
134
164
  next unless event
135
165
 
166
+ announce_resumed
167
+ watch_epoch(frame[:id]) if @epoch_watch
136
168
  @last_event_id = frame[:id] if frame[:id]
137
169
  yielded = true
138
170
  @on_event.call(event)
@@ -172,18 +204,55 @@ module Samagotchi
172
204
  def rediscover_worker
173
205
  return nil unless @rediscover
174
206
 
175
- found = @rediscover.call
207
+ found, reason = @rediscover.call
208
+ @reason = reason if reason
176
209
  return :gone if found == :gone
177
- return nil if found.nil? || found.port == @client.port
210
+ return nil if found.nil? || found.worker_key == @worker_key
178
211
 
179
212
  @client = found
180
213
  @last_event_id = nil
181
- @on_event.call({ "type" => "worker_changed", "port" => found.port }) unless closed?
214
+ moved(found.port, found.worker_key)
182
215
  :moved
183
216
  rescue StandardError
184
217
  nil
185
218
  end
186
219
 
220
+ # The frame +id+'s epoch (EventId: "<seq>-<epoch>") differs from the
221
+ # cursor's: a new worker answers the old cursor (its Bridge sends a
222
+ # reset next, or this is it). Announced before the frame.
223
+ def watch_epoch(id)
224
+ epoch = id.to_s[/-(\w+)\z/, 1]
225
+ cursor = @last_event_id.to_s[/-(\w+)\z/, 1]
226
+ return if epoch.nil? || cursor.nil? || epoch == cursor
227
+
228
+ @client.worker_key = epoch
229
+ moved(nil, epoch)
230
+ end
231
+
232
+ # Following another worker now: its key, and the worker_changed event
233
+ # (with its epoch for a stream through chi web).
234
+ def moved(port, key)
235
+ @worker_key = key
236
+ event = { "type" => "worker_changed", "port" => port }
237
+ event["epoch"] = key if @epoch_watch
238
+ @on_event.call(event) unless closed?
239
+ end
240
+
241
+ # A reconnect failed: said once, until a frame comes again.
242
+ def announce_retry
243
+ return if !@announce_retries || @retrying || closed?
244
+
245
+ @retrying = true
246
+ @on_event.call({ "type" => "stream_retrying" })
247
+ end
248
+
249
+ def announce_resumed
250
+ return unless @retrying
251
+
252
+ @retrying = false
253
+ @on_event.call({ "type" => "stream_resumed" })
254
+ end
255
+
187
256
  def finish(reason)
188
257
  @on_event.call({ "type" => "stream_closed", "reason" => reason }) unless closed?
189
258
  end
@@ -0,0 +1,77 @@
1
+ # frozen_string_literal: true
2
+
3
+ require "socket"
4
+
5
+ module Samagotchi
6
+ class BridgeClient
7
+ # The GET that opens an event stream, written by hand over a TCPSocket,
8
+ # and its response headers read up to the body: BridgeClient's stream
9
+ # (the worker's Bridge, `/session/:id/stream`), and any other server that
10
+ # serves the same SSE body at its own path with its own extra headers.
11
+ # TLS would plug in here.
12
+ module StreamRequest
13
+ # A reply's status line; its code is group 1 (BridgeClient::STATUS_LINE).
14
+ STATUS_LINE = %r{\AHTTP/1\.[01] (\d{3})}
15
+
16
+ module_function
17
+
18
+ # Open the stream and read past the response headers.
19
+ # @param host [String]
20
+ # @param port [Integer]
21
+ # @param path [String] the request path with its query
22
+ # @param last_event_id [String, nil] the reconnect cursor
23
+ # @param headers [Hash{String => String}] extra request headers, after
24
+ # Last-Event-ID
25
+ # @param timeout [Float, nil] give up when the headers aren't all in by then
26
+ # @return [Array(TCPSocket, Integer, Hash), nil] the socket positioned
27
+ # at the body, the HTTP status and the response headers (names in
28
+ # lower case), or nil when the server can't be reached
29
+ def open(host:, port:, path:, last_event_id: nil, headers: {}, timeout: nil)
30
+ sock = TCPSocket.new(host, port)
31
+ sock.write(request(host: host, port: port, path: path, last_event_id: last_event_id, headers: headers))
32
+ deadline = timeout && (Process.clock_gettime(Process::CLOCK_MONOTONIC) + timeout)
33
+ status = header_line(sock, deadline).to_s[STATUS_LINE, 1].to_i
34
+ received = {}
35
+ while (line = header_line(sock, deadline))
36
+ break if line.strip.empty?
37
+
38
+ name, value = line.split(":", 2)
39
+ received[name.strip.downcase] = value.to_s.strip if value
40
+ end
41
+ [sock, status, received]
42
+ rescue SystemCallError, SocketError, IOError
43
+ sock&.close rescue nil
44
+ nil
45
+ end
46
+
47
+ # The request's bytes.
48
+ # @return [String]
49
+ def request(host:, port:, path:, last_event_id: nil, headers: {})
50
+ lei = last_event_id.to_s.strip
51
+ last_event_line = lei.empty? ? "" : "Last-Event-ID: #{lei}\r\n"
52
+ extra = headers.map { |name, value| "#{name}: #{value}\r\n" }.join
53
+ "GET #{path} HTTP/1.1\r\nHost: #{host}:#{port}\r\nAccept: text/event-stream\r\n#{last_event_line}#{extra}" \
54
+ "Connection: keep-alive\r\n\r\n"
55
+ end
56
+
57
+ # One line of the response headers, or nil at EOF; raises
58
+ # Errno::ETIMEDOUT once +deadline+ (monotonic, nil: none) has passed. A
59
+ # line read in pieces, so a server that stops mid-line can't hold it.
60
+ def header_line(sock, deadline)
61
+ return sock.gets unless deadline
62
+
63
+ line = +""
64
+ loop do
65
+ remaining = deadline - Process.clock_gettime(Process::CLOCK_MONOTONIC)
66
+ raise Errno::ETIMEDOUT unless remaining.positive? && sock.wait_readable(remaining)
67
+
68
+ char = sock.getc
69
+ return line.empty? ? nil : line if char.nil?
70
+
71
+ line << char
72
+ return line if char == "\n"
73
+ end
74
+ end
75
+ end
76
+ end
77
+ end