chore(beads): remove the PinPoint-owned Beads server runtime - #2074
chore(beads): remove the PinPoint-owned Beads server runtime#2074timothyfroehlich wants to merge 5 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…only topology The original branch renamed scripts/beads-compatibility.json to beads-cloud-compatibility.json and described it as a snapshot of a dotfiles contract at all/.agents/beads/compatibility.json. That contract never landed (dotfiles PR #22 was closed unmerged), and the rename would have broken both dotfiles' drift test and scripts/tests/test_ci_change_detection.py, which name the file. The manifest stays where it was and remains the single source; the cloud scripts are restored to main's copy. Prose now describes what actually runs: beads is a local embedded Dolt database on the Mac with DoltHub as the sync remote, cloud routines install exactly the pinned versions, and the Mac's user-global mise pins are checked against this file by a dotfiles test. No Bazzite services exist. AGENTS.md's huddle paragraph pointed at ~/.agents/huddle/, which no longer exists. The huddle is now a public repository installed as a plugin, checked out at ~/Code/huddle; the Mac holds the leader role. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
timothyfroehlich
changed the title
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 985551c4c0
ℹ️ 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".
… runbook Codex review: the runbook's opening model still said concurrent cloud and live-server writes stop "the bridge", which fails closed for an operator. This branch deletes the bridge and the server, so the paragraph pointed at a recovery component that no longer exists. It now describes the topology that runs: DoltHub as hub, the Mac's embedded database pushing on auto-push and on every huddle poll, cloud routines cloning and pushing back before the sandbox ends. Conflicts are cell-level and surface on whichever side moves second — a rejected non-fast-forward push in the cloud, a failed pull on the Mac that also halts huddle rotation — and are resolved on the Mac, never with a forced push. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 988c1d04af
ℹ️ 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".
Codex review. inject-beads-actor.cjs still built the whoami path as ~/.agents/huddle/huddle-whoami.sh, and its test installed the stub at that same location, so the test kept passing after the directory was deleted. Since the huddle moved to its plugin checkout, every implicit Claude `bd` write in PinPoint has hit the fail-open catch and been stamped as the generic "Claude" instead of the session's registered identity. The hook now calls ~/Code/huddle/lib/huddle-whoami.sh — the checkout, not the harness plugin cache, because that path carries the plugin version. Verified against the live path: the hook resolves this session's registered name. The path is assembled with path.join, which is why a grep for the old string missed it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
timothyfroehlich
added
the
ready-for-review
1 participant
Summary
scripts/beads-server/(Dolt SQL server wrapper and unit, DoltHub bridge script, unit and timer,SETUP.md,server.yaml), the repository-local genericbeadsskill, and the tests that covered them. 962 lines out.scripts/beads-cloud-setup.sh,scripts/beads-cloud-init.sh, andscripts/beads-compatibility.jsonstay exactly as onmain. Cloud routines still clone from DoltHub and install exactly the pinnedbd/dolt.AGENTS.md's huddle paragraph now points at the publictimothyfroehlich/huddleplugin at~/Code/huddleinstead of the deleted~/.agents/huddle/.Reframed from the original draft. This branch was written for dotfiles PR #22, which would have moved the runtime into a dotfiles module and renamed the manifest to
beads-cloud-compatibility.jsonas a "snapshot" of it. #22 was closed unmerged and the two-host topology is gone, so the rename is dropped — it would also have broken dotfiles' drift test andscripts/tests/test_ci_change_detection.py, both of which name the file. The deletion is still wanted; the framing was not. Refreshed against currentmainwith a merge commit.Background: dotfiles#30, part of dotfiles#24.
Test Plan
pnpm run check— pass with pre-existing warningspnpm run check:python— 578 passedscripts/tests/test_beads_compatibility.py(integration-marked) plustest_ci_change_detection.pyandtest_check_skill_frontmatter.py— all passbeads-server,bd doctor,.agents/huddle,dolt-sql-server,dolthub-bridgeacross live files — none remain (historical records underdocs/superpowers/are left as records per AGENTS.md §8)Related Issues
Tracks PP-h2ui.29, whose "move into dotfiles" premise is superseded; the bead will be closed with that note when this merges.
—Claude-SingleHostMigration