Skip to content

chore(submodule): bump owletto to 1f740c5 (Mac menu-bar fixes)#840

Merged
buremba merged 1 commit into
mainfrom
fix/bump-owletto-local-init
May 18, 2026
Merged

chore(submodule): bump owletto to 1f740c5 (Mac menu-bar fixes)#840
buremba merged 1 commit into
mainfrom
fix/bump-owletto-local-init

Conversation

@buremba
Copy link
Copy Markdown
Member

@buremba buremba commented May 18, 2026

Summary

Submodule pointer bump only. No lobu-side code changes.

Picks up owletto#163 (1f740c5): Mac menu-bar fixes for the embedded-server flow.

  • User-confirmed adoption of external Lobu on :8787 — previously the menubar would silently POST `/api/local-init` against any process listening on the managed port, even one it didn't spawn. Now spawned-by-us still auto-adopts; an adopted external process surfaces a "Click Connect to adopt it" status and waits for an explicit click.
  • Authless-mode UX — when signed in via local-init bootstrap, hide the identity row + sign-out (the "Local Developer / dev@lobu.local" trio is the deployment itself) and drop the SF Symbol column from Connected Agents rows.
  • Selected-agent chip in popover header — "Idle · last poll 0s ago · Claude Code 2.1.143" alongside the sync status.
  • Section rename Watchers → Goals.
  • Pre-existing bug fixes that surfaced while wiring a manual goal:
    • `Watcher` struct expected JSON key `id` but server returns `watcher_id`. Decode silently failed forever; goals never rendered. Added CodingKeys mapping.
    • Goal filter compared `LobuWorkerIdentity.current()` ("mac-<uuid>") against `watcher.device_worker_id` (device-row UUID). Two different identifier spaces. Now resolves the row UUID via `/api/me/devices`.

Test plan

  • Mac app builds clean against owletto@1f740c5 in this worktree (`xcodebuild -scheme Lobu`).
  • e2e against PGlite `start-local.bundle.mjs`: adopted-external auto-launch does not POST local-init; explicit Connect click does; relaunch persists creds; GOALS section shows pinned watchers; agent chip renders in header.
  • CI: lint + typecheck on lobu-main pass with the new pointer.

Summary by CodeRabbit

  • Chores
    • Updated an internal subproject pointer to a newer commit, keeping internal dependencies current and ensuring build consistency.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ba68dd14-66d6-4ea1-9edd-669fdd7d4a27

📥 Commits

Reviewing files that changed from the base of the PR and between 0fa7a63 and d342fff.

📒 Files selected for processing (1)
  • packages/owletto

📝 Walkthrough

Walkthrough

The packages/owletto git submodule pointer is updated from one commit hash to another; no other code changes are present.

Changes

Submodule Update

Layer / File(s) Summary
Owletto submodule pointer update
packages/owletto
The packages/owletto subproject commit reference is updated to a new pinned hash.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • lobu-ai/lobu#825: Also updates the packages/owletto submodule pointer to a different pinned commit with no other code changes.

Poem

🐰 A submodule springs forth with glee,
New commits pinned, as they should be,
One small hash swap, clean and bright,
The owletto project takes its flight,
I nibble bytes and hop with delight.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically summarizes the main change—a submodule pointer bump for owletto with the key benefit (Mac menu-bar fixes) noted.
Description check ✅ Passed The description is comprehensive and follows the template structure with all critical sections completed: detailed summary of changes, test plan with checkmarks indicating what was validated, and relevant notes including the associated external PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bump-owletto-local-init

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

Picks up owletto-side menu-bar fixes:
  - User-confirmed adoption of external Lobu on :8787 (was: silently
    minting local-init creds against a process we didn't spawn).
  - Authless-mode UX: hide identity row + sign-out + agent SF Symbols.
  - Selected-agent chip in popover header.
  - Section rename Watchers → Goals.
  - Pre-existing decode bug fix (Watcher.id ← watcher_id) and the
    device-row UUID resolution for goal pinning.

No lobu-side code changes — submodule pointer move only.
@buremba buremba force-pushed the fix/bump-owletto-local-init branch from 0fa7a63 to d342fff Compare May 18, 2026 01:48
@buremba buremba merged commit f0d702c into main May 18, 2026
19 of 20 checks passed
@buremba buremba deleted the fix/bump-owletto-local-init branch May 18, 2026 01:49
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

2 participants