Skip to content

chore: bump owletto submodule (#196) + correct AGENTS.md guidance#926

Merged
buremba merged 2 commits into
mainfrom
feat/menubar-cancel-signin
May 19, 2026
Merged

chore: bump owletto submodule (#196) + correct AGENTS.md guidance#926
buremba merged 2 commits into
mainfrom
feat/menubar-cancel-signin

Conversation

@buremba
Copy link
Copy Markdown
Member

@buremba buremba commented May 19, 2026

Two related cleanups discovered while shipping owletto#196.

1. Bump owletto submodule pointer

Picks up owletto#196: feat(mac): Cancel button for the OAuth sign-in flow — three codex rounds of cancellation-race fixes; xcodebuild green on every commit.

Submodule SHA: 4f7c75705e966b.

2. AGENTS.md: correct the Mac/Xcode validation exception

  • The "E2E before merge" exception claimed agents can't reach Mac/Xcode. xcodebuild actually runs headlessly from Bash — verified during owletto#196's iteration. Reworded the exception so only UI/runtime validation is exempt; added the xcodebuild command to the validation table.
  • Updated "Cross-repo dispatch" to reference make task-setup workflow (which is how fix(ci): enable include-component-in-tag to fix auto-tagging #196 was shipped) instead of the retired standalone ~/Code/owletto clone.

Summary by CodeRabbit

Release Notes

This release contains internal updates and documentation improvements with no end-user visible changes.

  • Chores
    • Updated internal project references and developer documentation

Review Change Stack

Picks up:
- feat(mac): Cancel button for the OAuth sign-in flow (#196)

Before: 4f7c757 feat: per-agent guardrail toggle + recent trips view
After:  05e966b feat(mac): Cancel button for the OAuth sign-in flow
@buremba buremba enabled auto-merge (squash) May 19, 2026 17:19
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 76fbc369-8bb2-40e7-acb3-6ca67f54cd40

📥 Commits

Reviewing files that changed from the base of the PR and between 005eeca and 01f567d.

📒 Files selected for processing (1)
  • AGENTS.md
✅ Files skipped from review due to trivial changes (1)
  • AGENTS.md

📝 Walkthrough

Walkthrough

Bump packages/owletto subproject pointer to a new commit SHA and update AGENTS.md to use a .claude/worktrees/<slug>/packages/owletto worktree for cross-repo dispatch and add/clarify native/mac validation instructions.

Changes

Owletto submodule and documentation

Layer / File(s) Summary
Update packages/owletto submodule pointer
packages/owletto
The subproject commit pointer for packages/owletto is advanced to a new revision.
AGENTS.md: cross-repo dispatch and validation
AGENTS.md
Replaced owletto cross-repo dispatch instructions to use make task-setup NAME=<slug> with a .claude/worktrees/<slug>/packages/owletto branch checkout; clarified a validation exception for native/hardware UI workflows and added a packages/owletto/apps/mac/* xcodebuild validation entry.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • lobu-ai/lobu#866: Also updates the packages/owletto submodule commit SHA.
  • lobu-ai/lobu#886: Bumps the packages/owletto submodule pointer to another commit SHA.
  • lobu-ai/lobu#856: Contains a packages/owletto submodule update (related submodule-pointer change).

Poem

🐰 I nudged one pointer, soft and fleet,
A tiny bump — a tidy feat,
Docs now guide the worktree way,
Mac builds listed for the day,
Hooray — the repo hops along its beat!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides clear context for both changes with specific details (commit SHAs, owletto#196 reference, and rationale), but lacks the required test plan section from the template. Add a Test plan section documenting which validation steps were executed, as specified in the repository's description template.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes both main changes: bumping the owletto submodule and correcting AGENTS.md guidance, matching the actual changeset.
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/menubar-cancel-signin

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

Two-part fix discovered while shipping owletto#196:

1. The "E2E before merge" exception said agents can't reach
   Mac/Xcode dev environments. xcodebuild actually runs headlessly
   from Bash (verified: `xcodebuild -project Owletto.xcodeproj
   -scheme Owletto -configuration Debug -destination "platform=macOS"
   build CODE_SIGNING_ALLOWED=NO` produced "BUILD SUCCEEDED" on every
   commit of #196 without any human in the loop). Compile-checks
   aren't exempt; only UI/runtime validation is. Reworded the
   exception and added the xcodebuild command to the validation
   table so agents pick the right tool by package.

2. "Cross-repo dispatch" said owletto changes go through a standalone
   clone at ~/Code/owletto. That clone was retired this session in
   favour of `make task-setup NAME=<slug>` worktrees which set up
   packages/owletto on a real named branch under the worktree —
   matching how owletto#196 was actually shipped.
@buremba buremba changed the title chore: bump owletto submodule pointer (lobu-ai/owletto#196) chore: bump owletto submodule (#196) + correct AGENTS.md guidance May 19, 2026
@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