Skip to content

docs(status): mark fs broker as unwired - #164

Merged
0monish merged 1 commit into
mainfrom
agent/kel-175-native-status-truth
Sep 4, 2026
Merged

docs(status): mark fs broker as unwired#164
0monish merged 1 commit into
mainfrom
agent/kel-175-native-status-truth

Conversation

@0monish

@0monish 0monish commented Sep 4, 2026

Copy link
Copy Markdown
Member

Summary

  • Correct the canonical crate.keld-native status note: scoped filesystem handling is implemented, guard-checked, and wire-tested, but has no production host caller.
  • Remove the unsupported “live” product claim while preserving the row classification, evidence, target, issue, and verification SHA.
  • Regenerate the product-status view and agent corpus from the one TSV source.

Spec refs

  • KEL-175, atomic child of KEL-127 audit finding DG-01.
  • No boundary change: documentation truth only; broker, guard, host, permissions, IPC, and schema are unchanged.

Review gates

  • unsafe — none.
  • public API — none.
  • permission model — none.
  • dependency addition — none.
  • wire protocol — none.
  • CodeRabbit CLI 0.7.6 reviewed exact commit 9ac0f24 against 3bc5c5d with zero findings.

Tests

  • Negative census: serve_fs_session has no non-test caller anywhere under crates/; KEL-140 remains Backlog for host wiring.
  • just product-status, just product-status-check, just llms, just llms-check, and git diff --check passed.
  • Exact committed-tip isolated just ci — policy/generated-doc/Docker Mermaid gates, product-status 69/69, frozen TypeScript install/typecheck/41 tests, format, warning-denied workspace Clippy, 630/630 nextest with 2 intentional skips, rustdoc, and cargo-deny passed.

Platforms

  • Platform-neutral documentation correction; no real-OS product acceptance applies.
  • Hosted checks remain required before merge.

Perf impact

None.

@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: 02939ee4-c55c-4b1b-8501-4eb2058e8224

📥 Commits

Reviewing files that changed from the base of the PR and between 3bc5c5d and 9ac0f24.

⛔ Files ignored due to path filters (1)
  • docs/engineering/product-status.tsv is excluded by !**/*.tsv
📒 Files selected for processing (2)
  • docs/engineering/product-status.md
  • llms-full.txt

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@keldrobo keldrobo added size/XS <10 lines type:docs Documentation only labels Sep 4, 2026
@0monish
0monish merged commit 9f19d3d into main Sep 4, 2026
14 checks passed
@0monish
0monish deleted the agent/kel-175-native-status-truth branch September 4, 2026 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS <10 lines type:docs Documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants