Skip to content

docs(roadmap): add #322 #323 — json stream corruption and session identity contradiction#2838

Merged
Yeachan-Heo merged 1 commit intomainfrom
docs/roadmap-322-323-clean
Apr 29, 2026
Merged

docs(roadmap): add #322 #323 — json stream corruption and session identity contradiction#2838
Yeachan-Heo merged 1 commit intomainfrom
docs/roadmap-322-323-clean

Conversation

@code-yeongyu
Copy link
Copy Markdown
Collaborator

Pinpoints #322 and #323

ROADMAP.md only — no main.rs or test changes.

#322 — Config deprecation warnings corrupt --output-format json stream

eprintln!("warning: ...)inconfig.rs:~300emitted unconditionally duringClawSettings::load_merged()` regardless of output format. Combined stdout+stderr capture breaks JSON parse.

Fix shape: thread CliOutputFormat into config load; suppress prose warnings in json mode; inject as top-level "warnings": [...].

#323status --output-format json contradictory session identity

Same snapshot emits "session": "live-repl" (sentinel from session_path = None) alongside "session_lifecycle.kind": "saved_only" (tmux pane discovery). Two independent sources, no shared classification.

Fix shape: derive both fields from same lifecycle classification; replace "live-repl" sentinel with typed session_kind field.

Source: Jobdori live dogfood, mengmotaHost, 1da2781 main, 2026-04-29.

@Yeachan-Heo
Copy link
Copy Markdown
Contributor

OMX review session claw-code-pr-2838-review inspected the exact PR diff and CI state.

Verdict: APPROVE / merge-ready.

Scope verified:

  • ROADMAP.md only
  • ROADMAP.md | 3 +++
  • adds 合影留念 #322 JSON stream corruption by config deprecation warnings
  • adds 666 #323 contradictory status --output-format json session identity
  • confirmed no Rust / main.rs diff

Verification:

  • git diff --check origin/main...origin/pr-2838 passed
  • CI 4/4 green: docs source-of-truth, cargo fmt, cargo clippy --workspace, cargo test --workspace

No blockers found.


[repo owner's gaebal-gajae (clawdbot) 🦞]

@Yeachan-Heo Yeachan-Heo merged commit e7074f4 into main Apr 29, 2026
4 checks passed
@Yeachan-Heo Yeachan-Heo deleted the docs/roadmap-322-323-clean branch April 29, 2026 10:40
ChicoCifrado pushed a commit to ChicoCifrado/cifrado-code that referenced this pull request Apr 29, 2026
Constraint: Documentation-only follow-up from current main e7074f4 after PR ultraworkers#2838; edit scope limited to ROADMAP.md.\nRejected: Implementing provenance detection now | user requested roadmap entry only.\nConfidence: high\nScope-risk: narrow\nDirective: Future implementation should compare embedded build git_sha/build date to workspace HEAD/dirty state without leaking secrets.\nTested: git diff --check; scripts/fmt.sh --check\nNot-tested: Runtime provenance behavior; this commit only records the roadmap requirement.
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.

2 participants