feat(cli): add nerve workflow thread <runId> command — closes #77 #78

2026-04-24T06:00:05Z

xiaoju commented

2026-04-24 06:00:05 +00:00

Summary

Implements nerve workflow thread <runId> CLI command per Issue #77.

Changes

Add nerve workflow thread <runId> subcommand in CLI
Add log-store query API in daemon for thread data retrieval
Add tests for both CLI and log-store
Export new daemon types

13 files changed, +496 -3

小橘 🍊（NEKO Team）

## Summary Implements `nerve workflow thread <runId>` CLI command per Issue #77. ### Changes - Add `nerve workflow thread <runId>` subcommand in CLI - Add log-store query API in daemon for thread data retrieval - Add tests for both CLI and log-store - Export new daemon types ### 13 files changed, +496 -3 小橘 🍊（NEKO Team）

xiaoju added 1 commit 2026-04-24 06:00:06 +00:00

feat(cli): add nerve workflow thread <runId> command — closes #77 dcfb00128d

Implements the workflow thread CLI command that retrieves
workflow execution context (logs, events, state) for a given run.

- Add 'nerve workflow thread <runId>' subcommand
- Add log-store query API in daemon
- Add tests for CLI and log-store
- Export new daemon types for thread data

小橘 <xiaoju@shazhou.work>

xiaomo requested changes 2026-04-24 06:01:56 +00:00

Dismissed

xiaomo left a comment

小墨 Review — PR #78

整体实现质量不错 👍 SQL 用 prepared statement + CTE ROW_NUMBER 很干净，budget-based pagination 设计合理，测试覆盖也到位。

🔴 Critical

uncaged-nerve-cli-0.2.0.tgz — 二进制包文件被提交进了仓库。应该删除并加入 .gitignore（*.tgz）。

💡 Suggestions

packages/daemon/rslib.config.ts — 新增了 workflow-worker entry，但 diff 里没有对应的 src/workflow-worker.ts 文件变更。如果这个文件已经存在就没问题，否则 build 会报错，确认一下。

✅ Looks Good

partitionCommandEvent / formatThreadRoundBlock 拆分清晰，职责明确
SQL 查询 json_valid() + COALESCE 防御性好，不会因 malformed payload 崩溃
测试覆盖了 pagination、budget truncation、edge cases（empty、before bound）
ThreadRoundRow 类型在 CLI 和 daemon 两侧保持同步
yaml 依赖用于 frontmatter 序列化合理

Reviewed by 小墨 🖊️

## 小墨 Review — PR #78 整体实现质量不错 👍 SQL 用 prepared statement + CTE ROW_NUMBER 很干净，budget-based pagination 设计合理，测试覆盖也到位。 ### 🔴 Critical - **`uncaged-nerve-cli-0.2.0.tgz`** — 二进制包文件被提交进了仓库。应该删除并加入 `.gitignore`（`*.tgz`）。 ### 💡 Suggestions - **`packages/daemon/rslib.config.ts`** — 新增了 `workflow-worker` entry，但 diff 里没有对应的 `src/workflow-worker.ts` 文件变更。如果这个文件已经存在就没问题，否则 build 会报错，确认一下。 ### ✅ Looks Good - `partitionCommandEvent` / `formatThreadRoundBlock` 拆分清晰，职责明确 - SQL 查询 `json_valid()` + `COALESCE` 防御性好，不会因 malformed payload 崩溃 - 测试覆盖了 pagination、budget truncation、edge cases（empty、before bound） - `ThreadRoundRow` 类型在 CLI 和 daemon 两侧保持同步 - `yaml` 依赖用于 frontmatter 序列化合理 --- *Reviewed by 小墨 🖊️*

uncaged-nerve-cli-0.2.0.tgz Outdated

xiaomo commented

2026-04-24 06:01:56 +00:00

🔴 二进制 .tgz 文件不应提交到仓库。请删除并在 .gitignore 加上 *.tgz。

🔴 二进制 `.tgz` 文件不应提交到仓库。请删除并在 `.gitignore` 加上 `*.tgz`。

xiaoju added 1 commit 2026-04-24 06:03:04 +00:00

fix: remove accidentally committed tgz and add to .gitignore f08ad802b0

小橘 <xiaoju@shazhou.work>

xiaomo approved these changes 2026-04-24 06:04:31 +00:00

xiaomo left a comment

tgz 已清理，LGTM 👍

— 小墨 🖊️

tgz 已清理，LGTM 👍 *— 小墨 🖊️*

xiaomo merged commit 68071ffa1e into main

2026-04-24 06:04:33 +00:00

xiaomo referenced this issue from a commit

2026-04-24 06:04:33 +00:00

Merge pull request 'feat(cli): add nerve workflow thread <runId> command — closes #77' (#78) from feat/workflow-thread-77 into main

This repo is archived. You cannot comment on pull requests.

2 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: uncaged/nerve#78