Skip to content

chore(web): bump for connection device picker#717

Merged
buremba merged 4 commits into
mainfrom
feat/connection-device-picker
May 14, 2026
Merged

chore(web): bump for connection device picker#717
buremba merged 4 commits into
mainfrom
feat/connection-device-picker

Conversation

@buremba
Copy link
Copy Markdown
Member

@buremba buremba commented May 14, 2026

Bumps `packages/web` to pull in lobu-ai/owletto-web#106:

  • Editable Run on picker on the connection Settings tab (replaces the read-only "Pinned to" banner).
  • Sidebar device rows always render their pinned connections instead of only when the device is selected.

Test plan

  • Manual pin/unpin of X to MacBook via the new Settings-tab Select; DB `connections.device_worker_id` round-trips cleanly.
  • Sidebar shows Apple Screen Time + WhatsApp under the device without selecting it.

Summary by CodeRabbit

  • Chores
    • Updated an internal subproject pointer to keep project components in sync. This change is administrative only and does not alter functionality, public APIs, or exported interfaces.
    • No runtime behavior, user-facing features, or documentation were changed.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 14, 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: 060ea49f-1a7f-4798-b815-790b7bfbdc5d

📥 Commits

Reviewing files that changed from the base of the PR and between d63a456 and 29c51f4.

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

📝 Walkthrough

Walkthrough

This PR advances the packages/web subproject pointer from 4d9a17f1013be2f14fd0d1a50e5e529ca809000f to 117b62a8c0f7ebde2dec152e6daafb0515ceaf79.

Changes

Web Subproject Pointer Update

Layer / File(s) Summary
packages/web subproject commit reference
packages/web
The packages/web subproject pointer is updated from 4d9a17f1013be2f14fd0d1a50e5e529ca809000f to 117b62a8c0f7ebde2dec152e6daafb0515ceaf79.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • lobu-ai/lobu#711: Also updates the packages/web submodule pointer in a related change.

Poem

🐰
A tiny hop to bump the sha,
I twitch my nose and chew a paw.
New letters on the submodule line,
I stamp my foot — the tree looks fine.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: bumping the web package to include the connection device picker feature.
Description check ✅ Passed The description includes a clear summary of changes and a complete test plan with all items checked. However, it lacks a 'Notes' section, though this is optional.
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 feat/connection-device-picker

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

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 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 `@packages/web`:
- Line 1: The submodule pin for packages/web is set to a SHA that is not
reachable from owletto-web/main, causing check-drift to fail; update the
submodule entry at the top of the packages/web file to a commit SHA that is an
ancestor of owletto-web/main (or, if this PR is intentionally tracking a
different upstream branch, retarget the drift check branch used by check-drift
to that upstream), ensuring the new SHA is reachable from owletto-web/main and
re-run CI to confirm the drift check passes.
🪄 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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 7948faaa-544b-40f5-8bbe-d8305a13524a

📥 Commits

Reviewing files that changed from the base of the PR and between 24fdbb9 and 9ac71d7.

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

Comment thread packages/web Outdated
@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