Skip to content
This repository was archived by the owner on Aug 17, 2026. It is now read-only.

docs: codify Hive thin-adapter vision and guardrails - #50

Merged
100yenadmin merged 3 commits into
mainfrom
docs/49-thin-adapter-vision
Jul 27, 2026
Merged

docs: codify Hive thin-adapter vision and guardrails#50
100yenadmin merged 3 commits into
mainfrom
docs/49-thin-adapter-vision

Conversation

@100yenadmin

@100yenadmin 100yenadmin commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

  • preserve upstream VISION.md and add a fork-specific VISION_HIVE.md
  • define Hive as native Buzz plus Electric product identity, OAuth/company admission, additive settings, isolated per-company relay selection, and registered VM-agent adaptation
  • add a contribution runbook with authority boundaries, settings/relay/agent rules, upstream-update dispositions, patch-shape stop signals, and proof boundaries
  • add a minimal AGENTS.md pointer so Electric-only coding agents must read the guardrails

Architecture decisions

  • one isolated relay instance per company for the internal canary and guarded beta
  • one customer VM per company contains that company's relay and Hermes gateway; members use separate Hermes profiles
  • Hive consumes already-registered public VM-agent identities; it does not provision or configure Hermes
  • TARS, Samantha, and HAL 9000 may replace only the names, avatars, and default instructions of the native bundled starter personas
  • Electric configuration uses one additive section inside native Settings
  • invitations and seat changes remain in the Electric Sheep website People/Access flow for now
  • current support is customer-controlled screen sharing or remote control, with no permanent relay support identity
  • visible rebranding does not justify mass-renaming upstream crates, modules, protocols, or schemas
  • the target is a plugin-like boundary, not a new plugin framework

Validation

Proof boundary

This is a documentation-only change. It does not prove current source conformance, relay isolation, artifact/release readiness, VM-agent behavior, installed parity, or customer readiness. It changes no source behavior, schema, runtime, VM, Supabase, release, or customer state.

Closes #49

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 54 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 47ba8abb-689f-4ebd-9d33-9a40c638fc13

📥 Commits

Reviewing files that changed from the base of the PR and between d151f79 and f791e26.

📒 Files selected for processing (2)
  • VISION_HIVE.md
  • docs/hive-thin-adapter-runbook.md
📝 Walkthrough

Walkthrough

This documentation-only change adds Hive’s fork-specific vision, thin-adapter runbook, authority boundaries, isolation rules, review gates, and contributor-document routing. No exported entities or runtime behavior change. Confidence: 99%.

Changes

Hive thin-adapter governance

Layer / File(s) Summary
Hive vision and product boundaries
VISION_HIVE.md
Defines the Hive customer flow, owned seams, authority boundaries, per-company relay isolation, native-first behavior, agent/settings rules, support model, upstream patch discipline, and success criteria.
Thin-adapter contribution runbook
docs/hive-thin-adapter-runbook.md
Defines request classification, authority resolution, preferred extension seams, settings and relay rules, VM-agent constraints, and upstream update procedures.
Review and proof gates
docs/hive-thin-adapter-runbook.md
Adds stop signals, proposal questions, required issue/PR content, review dispositions, proof requirements, and first-principles constraints.
Contributor document routing
AGENTS.md
Requires Electric-only changes to follow the Hive vision and thin-adapter runbook while preserving upstream Buzz guidance.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related issues

  • #1 — The vision and runbook codify the thin-adapter architecture, authority boundaries, isolation rules, and VM-agent constraints described by the epic.
  • #4 — The documentation establishes constraints relevant to the OAuth and key-custody seam, but does not implement that authentication behavior.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PASS (96%): The title accurately and concisely describes the docs-only Hive vision and guardrails change.
Description check ✅ Passed PASS (95%): The description directly matches the documentation changes and linked issue scope.
Linked Issues check ✅ Passed PASS (94%): The docs implement the required Hive boundary, runbook, and AGENTS pointer without obvious requirement gaps.
Out of Scope Changes check ✅ Passed PASS (97%): The changes stay within documentation and routing updates, with no unrelated source or runtime modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/49-thin-adapter-vision

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

@100yenadmin

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai coderabbitai 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.

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/hive-thin-adapter-runbook.md`:
- Around line 102-108: Update the relay and tenant-selection rules in the
runbook to state that the URL host is authoritative for community binding; OAuth
or token claims may authenticate or narrow access but must never override the
host-derived community. Require rejecting any host/claim mismatch before
admission, while preserving the existing server-side selection and fail-closed
behavior.

In `@VISION_HIVE.md`:
- Around line 30-32: Update the signed Hive update-channel requirements in
VISION_HIVE.md and its authority table to assign ownership for the signer, key
custody, rotation, and revocation; identify the artifact-verification gate and
required release/install verification evidence; and document failure behavior
when signature verification fails. Ensure the documented ownership and evidence
prove the signed-updates invariant.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 68cce660-4966-411c-8455-de4e3a43a3d1

📥 Commits

Reviewing files that changed from the base of the PR and between 6d5c0af and d151f79.

📒 Files selected for processing (3)
  • AGENTS.md
  • VISION_HIVE.md
  • docs/hive-thin-adapter-runbook.md
📜 Review details
⏰ Context from checks skipped due to timeout. (10)
  • GitHub Check: Desktop E2E Integration (2/2)
  • GitHub Check: Desktop E2E Integration (1/2)
  • GitHub Check: Desktop Smoke E2E (4)
  • GitHub Check: Desktop Core
  • GitHub Check: Desktop Build (macOS)
  • GitHub Check: Desktop Smoke E2E (3)
  • GitHub Check: Desktop Smoke E2E (1)
  • GitHub Check: Desktop Smoke E2E (2)
  • GitHub Check: Analyze (javascript-typescript)
  • GitHub Check: Analyze (rust)
🧰 Additional context used
📓 Path-based instructions (1)
**/AGENTS.md

📄 CodeRabbit inference engine (CLAUDE.md)

Follow the instructions documented in each AGENTS.md file for the corresponding directory and its descendants.

Files:

  • AGENTS.md
🧠 Learnings (1)
📓 Common learnings
Learnt from: CR
Repo: electricsheephq/evaOS-Hive

Timestamp: 2026-07-27T07:55:51.194Z
Learning: Run `just ci` before every pull request; run `just test` for integration changes to `buzz-relay`, `buzz-db`, or `buzz-auth`.
Learnt from: CR
Repo: electricsheephq/evaOS-Hive

Timestamp: 2026-07-27T07:55:51.194Z
Learning: Prefer modeling new feature operations as Nostr events rather than adding HTTP endpoints; add HTTP only for explicitly HTTP-only surfaces such as media, webhooks, git smart HTTP, metadata, health checks, or generic Nostr bridge endpoints.
🪛 LanguageTool
VISION_HIVE.md

[style] ~167-~167: To elevate your writing, try using more formal phrasing here.
Context: ...ails. The proof is that native features keep working, the Electric patch stack stays small a...

(CONTINUE_TO_VB)

🔇 Additional comments (1)
AGENTS.md (1)

7-11: LGTM!

Comment thread docs/hive-thin-adapter-runbook.md
Comment thread VISION_HIVE.md

@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: f791e26345

ℹ️ 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".

Comment thread VISION_HIVE.md
Comment on lines +69 to +70
The signed-update gate is the exact-tag release workflow in
[`release.yml`](.github/workflows/release.yml). Release evidence must bind the

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 Point Hive releases at an executable gate

When a Hive release candidate follows this mandatory guide, the linked workflow cannot act as its release gate: .github/workflows/release.yml restricts its setup and platform jobs to github.repository == 'block/buzz' and publishes to block/buzz's buzz-desktop-latest, while the Hive client expects electricsheephq/evaOS-Hive's hive-desktop-latest. Tagging this fork therefore skips the release jobs rather than producing the asserted Hive artifact and signature evidence; point this section to a Hive-capable workflow or implement that gate before declaring it authoritative.

AGENTS.md reference: AGENTS.md:L7-L10

Useful? React with 👍 / 👎.

@100yenadmin
100yenadmin merged commit d647f76 into main Jul 27, 2026
30 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: codify the Hive thin-adapter vision and contribution guardrails

1 participant