chore(scripts): task-clean repo resolution, drop mac symlink, surface PORT

[buremba]

Summary

Three small dev-script fixes for the per-task worktree workflow, scoped by a codex review of the multi-port story.

• task-clean.sh repo resolution. Running from inside a worktree resolved repo to that worktree, so the worktree-dir + branch deletions targeted the wrong place. Switch to git rev-parse --git-common-dir --path-format=absolute, matching the fix task-setup.sh (fix(cli): task-setup uses --path-format=absolute for git-common-dir #900) and task-use.sh already use.
• task-clean.sh warn on lobu context rm failure. Mirrors task-setup.sh:188. Silent failures left stale context entries after a clean.
• task-use.sh drops the Mac symlink. The Mac menubar reads ~/.config/lobu/config.json on every popover expand (AppState.swift:456 refreshContexts) and shows each context's apiUrl independently — per-worktree contexts already appear in its picker, no "active mac" indirection needed. Devs open Xcode at the worktree's packages/owletto/apps/mac directly. The Chrome symlink stays.
• task-use.sh surfaces PORT. Reads PORT from .env.local and prints it in the success message. The Chrome extension's gateway URL (sidepanel "Server URL" → chrome.storage.local) and the source symlink are independent switches; surfacing PORT each invocation makes the "still pointed at the previous worktree" footgun visible.

Reproducer

$ cd /Users/burakemre/Code/lobu/.claude/worktrees/task-script-fixes
$ bash scripts/task-use.sh task-script-fixes
→ chrome: /Users/burakemre/.config/lobu-dev/active/chrome → …/task-script-fixes/packages/owletto/apps/chrome
✓ active worktree: task-script-fixes
  ↳ Chrome ext: set Server URL to http://localhost:8789
    (open the owletto sidepanel; or reset via chrome://extensions → Reload)
    MV3 service workers re-register on extension reload, not on symlink change.

task-clean.sh repo-resolution path was syntax-checked (bash -n); not run end-to-end because doing so would remove this worktree mid-PR. The change is mechanical (one variable, identical to existing task-setup.sh / task-use.sh pattern).

Notes

• Per-port auth cookies. Each localhost:$PORT has its own better-auth session — expected, but worth knowing when switching worktrees. Not changing.
• Menubar refresh cadence. refreshContexts() is on-popover, not a filesystem watcher; new contexts appear on the next menu open. Acceptable.

Test plan

• bash -n scripts/task-{clean,use}.sh
• bash scripts/task-use.sh <worktree> retargets chrome symlink + prints PORT
• Verify next task-setup + task-clean round-trip after merge (uses the new repo resolution from inside a worktree)

Summary by CodeRabbit

• Improvements
  • Enhanced error handling in cleanup operations with explicit warnings for missing dependencies.
  • Streamlined extension development setup process.
  • Improved setup guidance for configuring extension server connection and MV3 reload behavior.

[coderabbitai]

📝 Walkthrough

Walkthrough

Both scripts/task-clean.sh and scripts/task-use.sh are updated to use Git-based directory resolution. task-clean.sh adds explicit error handling for Lobu context cleanup. task-use.sh removes Mac symlink setup, retargets Chrome only, and reads active worktree PORT from .env.local to guide users in setting the Chrome extension "Server URL".

Changes

Task Script Robustness and Symlink Updates

Layer / File(s)	Summary
task-clean.sh: repo path resolution and error handling scripts/task-clean.sh	repo is resolved from the script's location via git rev-parse --path-format=absolute --git-common-dir instead of cd "$script_dir/..". Lobu context removal explicitly warns to stderr if the command fails or if lobu is not on PATH.
task-use.sh: Chrome symlink retargeting and port-aware user guidance scripts/task-use.sh	Mac symlink setup is removed; only the Chrome ~/.config/lobu-dev/active/chrome symlink is updated. Header docs clarify that the extension "Server URL" must be set manually. When using a worktree, PORT is parsed from $source_root/.env.local and printed in instructions to set the Chrome sidepanel to http://localhost:<PORT>. Repo resolution comment reframes the Git shared .git path requirement.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• lobu-ai/lobu#909: Modifies scripts/task-use.sh around repo/worktree path resolution, switching to git rev-parse ... --git-common-dir for symlink retargeting.
• lobu-ai/lobu#900: Updates scripts' repo path resolution to use git rev-parse --path-format=absolute --git-common-dir to avoid cwd-dependent worktree path bugs.
• lobu-ai/lobu#899: Updates task scripts' repo-root resolution to use git rev-parse --git-common-dir instead of cd "$script_dir/..", aligning shared-worktree path logic.

Poem

🐰 Our task scripts now resolve their paths with Git's sure hand,
No more fumbling through symlinks in this scattered land;
Chrome gets its retarget, and warnings are clear,
PORT reads from .env, bringing guidance near!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the three main changes: task-clean repo resolution fix, removal of mac symlink, and surfacing of PORT in task-use.sh.
Description check	✅ Passed	The PR description is comprehensive with detailed summaries of changes, a reproducer example, and test plan; it deviates from the template structure but provides all essential information in an organized manner.
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 docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/task-script-fixes

---

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

[codecov-commenter]

⚠️ Please install the 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!

[coderabbitai]

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 `@scripts/task-use.sh`:
- Around line 91-94: The PORT parsing currently assigned to active_port from
env_local can return quoted or commented values (e.g. PORT="5173" or PORT=5173 #
dev); update the parsing logic around active_port to robustly extract a numeric
port: read the PORT line (grep or awk matching ^PORT=), strip surrounding
quotes, remove any trailing inline comments or whitespace, and validate/retain
only the digits (fall back to empty or a default if not numeric). Target the
code that sets env_local and active_port so the updated command normalizes
values like "5173" and ignores comments before assigning active_port.

🪄 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: 2dd8c5e5-e3da-4f90-84a7-2c989fa9d142

📥 Commits

Reviewing files that changed from the base of the PR and between f8f087b and 0f827e9.

📒 Files selected for processing (2)

• scripts/task-clean.sh
• scripts/task-use.sh

[coderabbitai]

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Harden PORT parsing from .env.local to handle common formats.

Line 93 only matches PORT=... and can emit bad output for quoted/commented values (for example PORT="5173" or PORT=5173 # dev).

Suggested fix

-    active_port="$(awk -F= '/^PORT=/{print $2; exit}' "$env_local" | tr -d '[:space:]')"
+    active_port="$(sed -nE 's/^[[:space:]]*PORT[[:space:]]*=[[:space:]]*"?([^"#[:space:]]+)"?.*$/\1/p' "$env_local" | head -n1)"

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	env_local="$source_root/.env.local"
	if [[ -f "$env_local" ]]; then
	active_port="$(awk -F= '/^PORT=/{print $2; exit}' "$env_local" | tr -d '[:space:]')"
	fi
	env_local="$source_root/.env.local"
	if [[ -f "$env_local" ]]; then
	active_port="$(sed -nE 's/^[[:space:]]*PORT[[:space:]]*=[[:space:]]*"?([^"#[:space:]]+)"?.*$/\1/p' "$env_local" | head -n1)"
	fi

🤖 Prompt for 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.

In `@scripts/task-use.sh` around lines 91 - 94, The PORT parsing currently
assigned to active_port from env_local can return quoted or commented values
(e.g. PORT="5173" or PORT=5173 # dev); update the parsing logic around
active_port to robustly extract a numeric port: read the PORT line (grep or awk
matching ^PORT=), strip surrounding quotes, remove any trailing inline comments
or whitespace, and validate/retain only the digits (fall back to empty or a
default if not numeric). Target the code that sets env_local and active_port so
the updated command normalizes values like "5173" and ignores comments before
assigning active_port.