diff options
| author | Craig Jennings <c@cjennings.net> | 2026-07-24 17:42:54 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-07-24 17:42:54 -0500 |
| commit | b19d420b11af77e991b4bf14ef5a312c70cbfa3e (patch) | |
| tree | cbda5af7f2afd1d1523a65b002f4a1aaa48c5ad6 /working/triage-telegram-down-launch/note-superseded-1723.txt | |
| parent | f0c1bc40708615d5b423922c08c1f27e6cf96259 (diff) | |
| download | rulesets-b19d420b11af77e991b4bf14ef5a312c70cbfa3e.tar.gz rulesets-b19d420b11af77e991b4bf14ef5a312c70cbfa3e.zip | |
chore(inbox): park lint-org and telegram fixes from home and .emacs.d
Diffstat (limited to 'working/triage-telegram-down-launch/note-superseded-1723.txt')
| -rw-r--r-- | working/triage-telegram-down-launch/note-superseded-1723.txt | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/working/triage-telegram-down-launch/note-superseded-1723.txt b/working/triage-telegram-down-launch/note-superseded-1723.txt new file mode 100644 index 0000000..7c54ede --- /dev/null +++ b/working/triage-telegram-down-launch/note-superseded-1723.txt @@ -0,0 +1,22 @@ +Fix for a defect in triage-intake.telegram.org: the numbered Step 1 was missing the mandatory `(setq telega-use-docker t)` that the plugin's own Quick Reference and SEGFAULT gotcha both require. Attached is the edited workflow file (.ai/workflows/triage-intake.telegram.org) — please take it into the canonical at claude-templates/.ai/workflows/. + +THE DEFECT + +The plugin contradicts itself across three places: +- Quick Reference (line 35): `emacsclient -e "(progn (setq telega-use-docker t) (telega t) 'started)"` — has the setq. +- SEGFAULT gotcha (line ~170): "docker mode stays mandatory (telega-use-docker = t; the setq before (telega t) is still the right defense)". +- Numbered Step 1 (lines 91-93): `(progn (unless (...live-p) (telega t)) 'started)` — NO setq. + +telega-use-docker defaults to nil. In native (non-docker) mode the dockerized-vs-native tdlib difference is exactly the SEGFAULT the gotcha documents (exit 139). A session following the numbered Step 1 literally, on a daemon where nothing had already forced telega-use-docker to t, starts telega native and crashes the server — which the engine reports as SCAN FAILED at the top of the summary. Both of Craig's personal projects that use the telegram source (work and home) reported triage-intake failing. + +THE FIX + +Added `(setq telega-use-docker t)` as the first form in Step 1's progn, before the `(unless ... (telega t))`, with a comment pointing at the gotcha. Now Step 1 matches the Quick Reference. Minimal, wording/robustness only; no behavior change on a daemon that already had docker mode on. + +IMPORTANT CAVEAT — this is a real defect but NOT a confirmed root cause. I could not reproduce the original failure: Craig doesn't remember the symptom ("it was much earlier"), and his .emacs.d daemon currently reads telega-use-docker t (via .emacs.d's telega-config `:custom`), so on his machine right now the missing setq may have been moot. The fix removes one genuine failure mode that matches the reported symptom; whether it was THE cause is unconfirmed. I've asked work and home for their actual error via their inboxes; if they come back with something else (e.g. the recent :TRIAGE_SOURCES: gating change 4d87f35, or a different source), I'll send a follow-up. + +Two things worth your judgment on the canonical: +1. The stale note at line 37 ("Craig's daemon currently has telega-use-docker nil") is now false on .emacs.d — telega-config sets it t. Consider softening it to "the daemon's default is nil unless an Emacs-config :custom forces it," since the workflow syncs to machines/daemons without that config. +2. If the daemon reliably has docker mode on everywhere telega runs, this whole class is belt-and-braces — but Step 1 contradicting the Quick Reference is a defect regardless, and the belt is cheap. + +No reply needed unless you disagree with the fix. |
