Skip to content

kb init(jiuwenswarm): modules - #311

Closed
tzhouam wants to merge 1 commit into
mainfrom
kb/init-jiuwenswarm-modules-newinit-20261005-v2
Closed

tzhouam wants to merge 1 commit into
mainfrom
kb/init-jiuwenswarm-modules-newinit-20261005-v2

Conversation

@tzhouam

@tzhouam tzhouam commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

kb init stage modules for jiuwenswarm (openJiuwen-ai/jiuwenswarm).

  • Upstream pin: f0a69728c96b5961d993449f1a901cbd2f4dac5b
  • Knowledge base: 590fd45f7edf0c99fc16ea163b5a179f7570d384
  • Model spend (accounted): $0.00

Human-merged. Rules were screened by the docs redundancy filter, checked at the pin, and given an advisory verdict; fail rules are already removed.

Coverage:

metric before after
modules routed 87.5% 100.0%

Needs human edit:

  • source index docs/assets/images/config-model-test.png: read failed; retained as unknown
  • source index docs/assets/images/config-vision-model-test.png: read failed; retained as unknown
  • source index docs/assets/images/config_model_connect_test.png: read failed; retained as unknown
  • source index jiuwenswarm/channels/web/frontend/tests/fixtures/trajectory-archive-subjects.trajectory.zip: read failed; retained as unknown
  • source index tests/unit_tests/agentserver/test_build_config_entry_transport.py: parse failed; retained as unknown
  • source index tests/unit_tests/gateway/test_channel_manager_file_fanout.py: parse failed; retained as unknown

Notes:

  • unlimited subscription generation and independent judgment explicitly selected; no stage USD ceiling applies; fixed USD accounting is observability only, and unreported invoiced costs remain unknown
  • module jiuwenswarm/channels/browser/: absorbed into owner browser-client (jiuwenswarm/channels/browser/)
  • module jiuwenswarm/channels/ide/: absorbed into owner ide-clients (jiuwenswarm/channels/ide/packages/jetbrains-plugin/, jiuwenswarm/channels/ide/packages/shared-webview/, jiuwenswarm/channels/ide/packages/vscode-extension/esbuild.js, jiuwenswarm/channels/ide/packages/vscode-extension/src/client/, jiuwenswarm/channels/ide/packages/vscode-extension/src/codeActions/, jiuwenswarm/channels/ide/packages/vscode-extension/src/context/, jiuwenswarm/channels/ide/packages/vscode-extension/src/editor/, jiuwenswarm/channels/ide/packages/vscode-extension/src/swarm/, jiuwenswarm/channels/ide/packages/vscode-extension/src/ui/StatusBar.ts)
  • module jiuwenswarm/channels/tui/: absorbed into owner tui-client (jiuwenswarm/channels/tui/)
  • module jiuwenswarm/extensions/: absorbed into owner extensions-plugins (jiuwenswarm/extensions/init.py)
  • module jiuwenswarm/runtime/: absorbed into owner agent-runtime (jiuwenswarm/runtime/init.py)
  • module scripts/: 28 prefixes would be needed to absorb it into owner protocols; it gets its own map card

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-05T20:21:51.775881Z c67b1dd PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c67b1ddaf5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- jiuwenswarm/common/acp/
- jiuwenswarm/common/config_panel/
- jiuwenswarm/common/schema/
- jiuwenswarm/scripts/

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Route the WhatsApp bridge to gateway-channels

When a PR changes jiuwenswarm/scripts/whatsapp-bridge.js, this directory-wide prefix makes Direct return only common-core; however, components/gateway-channels/feature-depth-im-whatsapp.md documents that script as the runtime bridge started by WhatsAppChannel, including its lifecycle and configuration contracts. Route that file to gateway-channels and reserve common-core for the unrelated folder-generation scripts so reviews receive the relevant failure-mode knowledge.

AGENTS.md reference: AGENTS.md:L12-L17

Useful? React with 👍 / 👎.

Comment on lines +555 to +557
- jiuwenbox/src/jiuwenbox/proxy/inference_privacy_proxy.py
- jiuwenbox/src/jiuwenbox/proxy/inference_privacy_proxy_manager.py
- jiuwenbox/src/jiuwenbox/server/app.py

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Keep broad subpackage routes from consuming the cap

These new narrow routes overlap the pre-existing subpackages prefixes (jiuwenbox/ and sdks/). For a descriptionless PR changing the sandbox app, SDK client, and TUI, Direct ranks the generic subpackages route first, then adds sandbox-runtime and sdk-clients, exhausting its three-route cap and omitting tui-client even though that owner was touched. Remove or narrow the generic coverage, or otherwise ensure each file resolves to its nearest owner without consuming duplicate route slots.

AGENTS.md reference: AGENTS.md:L11-L17

Useful? React with 👍 / 👎.

Comment on lines +19 to +20
- [scripts/ — 打包与构建脚本](scripts.md)
- [NFS 部署脚本(scripts/nfs)](scripts-nfs.md)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Merge script summaries into the existing indexed pages

These links create scripts.md and scripts-nfs.md alongside the already-linked scripts/_index.md and scripts-nfs/_index.md, duplicating the same module summaries and source contracts under two canonical-looking destinations. Future edits can leave the parallel pages inconsistent; merge the new architecture material into the existing indexed destinations instead of maintaining synonymous pages.

AGENTS.md reference: AGENTS.md:L7-L11

Useful? React with 👍 / 👎.

@tzhouam tzhouam closed this Oct 5, 2026
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.

1 participant