Skip to content

chore: bump web submodule for #117 (CI unblock)#728

Merged
buremba merged 1 commit into
mainfrom
chore/bump-web-uselocation-fix
May 14, 2026
Merged

chore: bump web submodule for #117 (CI unblock)#728
buremba merged 1 commit into
mainfrom
chore/bump-web-uselocation-fix

Conversation

@buremba
Copy link
Copy Markdown
Member

@buremba buremba commented May 14, 2026

Summary

Picks up lobu-ai/owletto-web#117 which fixes the two pre-existing TS errors that have been blocking Build and Push Images on every main push since the UX sweep:

  • useLocation was imported + called in events-tab/index.tsx but the value was unused. Dropped.
  • z.record(z.unknown()) is a type + runtime error on the pinned zod 4.4.3 (API now requires (keyType, valueType)). Replaced with z.record(z.string(), z.unknown()).

After this lands, the next push to main should successfully build and push an image, allowing the deploy pipeline to roll forward with #725, #726, and #727 (action_modes).

Test plan

  • cd packages/web && bun run build passes locally.
  • Build and Push Images succeeds on main after merge.

Summary by CodeRabbit

  • Chores
    • Updated web package subproject reference to the latest version.

Review Change Stack

Picks up lobu-ai/owletto#117 which deletes an unused useLocation
call and fixes z.record(z.unknown()) → z.record(z.string(), z.unknown())
on the pinned zod 4.4.3. Together they were the two failing TS errors
that have been blocking Build and Push Images on every main push since
the UX sweep landed.
@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: abe2aebe-ea51-40b4-a7a7-cd2959d96e78

📥 Commits

Reviewing files that changed from the base of the PR and between 7fe865e and bc9722b.

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

📝 Walkthrough

Walkthrough

The packages/web Git submodule reference is updated to point to a new commit hash, pinning the web package to a different revision without any application logic changes in this repository.

Changes

Web Submodule Update

Layer / File(s) Summary
Web submodule pointer update
packages/web
Submodule commit reference updated to a new revision.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • lobu-ai/lobu#718: Updates the same packages/web submodule commit hash reference.
  • lobu-ai/lobu#714: Includes a web submodule reference update as part of the changes.
  • lobu-ai/lobu#717: Updates the packages/web Git subproject pointer in the same way.

Poem

🐰 A small hop down the dependency lane,
Web packages dance in the submodule chain,
A commit hash changed, pinned tight and true,
The rabbit hops on to what's fresh and new! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-web-uselocation-fix

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

@buremba buremba merged commit 6f07b3d into main May 14, 2026
15 of 18 checks passed
@buremba buremba deleted the chore/bump-web-uselocation-fix branch May 14, 2026 20:41
@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