feat(cli): workspace biome.json with noConsole, remove dryRun console.log #107

Merged
xiaomo merged 1 commits from feat/106-workspace-biome into main 2026-04-25 01:15:53 +00:00
Owner

What

Workspace init 生成 biome.json 禁止 console,同时去掉 workflow-utils 里 dryRun 的 console.log。

Why

Workflow 代码的 stdout 无持久化(daemon worker stdio 是 inherit),console.log 没人看。有意义的数据走 role 返回值进 log-store。

Changes

  • cli/commands/init.ts: 新增 BIOME_JSON 模板(noConsole: error),nerve init 时写入 workspace;PACKAGE_JSON 模板加 @biomejs/biome
  • workflow-utils: spawnSafecursorAgentllmExtract 去掉 dryRun 的 console.log,保留 stub 返回
  • tests: 去掉对应的 console.log spy/mock

Tests

378 tests pass

Ref

Fixes #106

## What Workspace init 生成 biome.json 禁止 console,同时去掉 workflow-utils 里 dryRun 的 console.log。 ## Why Workflow 代码的 stdout 无持久化(daemon worker stdio 是 inherit),console.log 没人看。有意义的数据走 role 返回值进 log-store。 ## Changes - **cli/commands/init.ts**: 新增 `BIOME_JSON` 模板(`noConsole: error`),`nerve init` 时写入 workspace;`PACKAGE_JSON` 模板加 `@biomejs/biome` - **workflow-utils**: `spawnSafe`、`cursorAgent`、`llmExtract` 去掉 dryRun 的 `console.log`,保留 stub 返回 - **tests**: 去掉对应的 console.log spy/mock ## Tests 378 tests pass ## Ref Fixes #106
xiaoju added 1 commit 2026-04-25 01:01:31 +00:00
- init.ts: scaffold biome.json with noConsole: error for workflows
- package.json template: add @biomejs/biome to devDependencies
- workflow-utils: remove console.log from dryRun paths (stub returns
  are captured by log-store via role results)

Fixes #106
xiaomo approved these changes 2026-04-25 01:15:52 +00:00
xiaomo left a comment
Owner

LGTM

增量很干净:

  1. nerve init 生成 biome.json(noConsole: error)+ devDep 加 @biomejs/biome — 新 workspace 开箱即用
  2. workflow-utils 三处 dryRun 去掉 console.log,与 noConsole 规则对齐,测试也移除了 logSpy mock

合并 🚀

✅ LGTM 增量很干净: 1. `nerve init` 生成 biome.json(noConsole: error)+ devDep 加 @biomejs/biome — 新 workspace 开箱即用 2. workflow-utils 三处 dryRun 去掉 console.log,与 noConsole 规则对齐,测试也移除了 logSpy mock 合并 🚀
xiaomo merged commit 7ce3970027 into main 2026-04-25 01:15:53 +00:00
This repo is archived. You cannot comment on pull requests.
No Reviewers
No Label
2 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/nerve#107