clawdie/colibri
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 33

chore/store-phase3-followups #228

Merged
clawdie merged 2 commits from chore/store-phase3-followups into main 2026-06-26 22:12:12 +02:00
Conversation 0 Commits 2 Files changed 4 +140 -16
clawdie commented 2026-06-26 22:11:56 +02:00
Owner
Copy link
No description provided.
clawdie added 2 commits 2026-06-26 22:11:59 +02:00 
The Phase 3 landing (host/last_seen/heartbeat/migrations) shipped with
zero test coverage — test_agent_registry only registered with host=None
and never exercised the new surface. Add 5 tests:

- register_agent persists host + stamps last_seen
- heartbeat advances last_seen and preserves host via COALESCE (None)
- heartbeat(Some(host)) updates host in place
- heartbeat on unknown agent id fails closed (StoreError::NotFound)
- run_migrations idempotent across store reopen (file-based; the ALTER
  statements must not raise "duplicate column name" on the second open)

Store crate: 11 -> 16 tests, all green.

(Sam & Claude)
style: rustfmt drift from Phase 3 landing
Some checks are pending
CI / rust (pull_request) Waiting to run
Details
CI / markdown (pull_request) Waiting to run
Details
CI / port (pull_request) Waiting to run
Details
CI / agent-jail-pkgs (pull_request) Waiting to run
Details
1e4e2f9368 
The host/last_seen fields added in ba5ee66 left mis-indented struct
literals in scheduler.rs tests and a cramped match arm in socket.rs.
cargo fmt --all (no runner in CI catches this; see quality-gates.md).

(Sam & Claude)
clawdie merged commit c35bbd273a into main 2026-06-26 22:12:12 +02:00
clawdie deleted branch chore/store-phase3-followups 2026-06-26 22:12:15 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
first-proof blocker

gates the first proven end-to-end

  
hardening

security/robustness follow-up, not a first-proof gate

No labels
first-proof blocker
hardening
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: clawdie/colibri#228
No description provided.
Delete branch "chore/store-phase3-followups"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?