Skip to content

fix(browser-use): load AppKit framework before hiding macOS dock icon#1

Closed
erudenko wants to merge 1 commit intomainfrom
fix/browser-use-dock-icon-v1.0.2
Closed

fix(browser-use): load AppKit framework before hiding macOS dock icon#1
erudenko wants to merge 1 commit intomainfrom
fix/browser-use-dock-icon-v1.0.2

Conversation

@erudenko
Copy link
Copy Markdown
Member

@erudenko erudenko commented Mar 3, 2026

Summary

  • The v1.0.1 dock icon fix was silently failing — NSApplication class isn't available in a bare Python process because AppKit framework isn't loaded
  • Added ctypes.cdll.LoadLibrary("/System/Library/Frameworks/AppKit.framework/AppKit") before objc_getClass(b"NSApplication")
  • objc_getClass returned NULL → objc_msgSend(NULL, ...) was a no-op → the entire fix chain did nothing
  • Bumps browser-use to v1.0.2

Test plan

  • Verified NSApplication class resolves to non-None after loading AppKit
  • Verified setActivationPolicy: returns policy=2 (Prohibited)
  • Patched cached v1.0.0 and v1.0.1 for immediate local testing

The previous dock icon fix was silently failing because NSApplication
class isn't available in a bare Python process — AppKit must be loaded
first. Added ctypes.cdll.LoadLibrary for AppKit.framework before
objc_getClass(b"NSApplication").

- Root cause: objc_getClass returns NULL without AppKit loaded
- objc_msgSend(NULL, ...) is a no-op, so the fix silently did nothing
- Bump to v1.0.2
@erudenko erudenko closed this Mar 3, 2026
@erudenko erudenko deleted the fix/browser-use-dock-icon-v1.0.2 branch March 3, 2026 11:36
erudenko added a commit that referenced this pull request Mar 23, 2026
…display

- Rename /gtd:process → /gtd:clarify (GTD step 2)
- Rename /gtd:work → /gtd:engage (GTD step 5)
- Add "reference" list type for non-actionable items worth keeping
- Replace random hex IDs (gtd-a1b2c3) with sequential numbers (#1, #2, #3)
- Add gtd-display.ts Bun tool for all terminal output (replaces inline echo -e)
- Boxed output with unicode borders, background-colored title bars, overflow truncation
- Workflow now matches canonical GTD: Capture → Clarify → Organize → Reflect → Engage
- Store schema v2.0 with nextId counter for atomic ID allocation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
erudenko added a commit that referenced this pull request Mar 31, 2026
…n version sync

Three root causes for persistent Test Plugins CI failures:

1. pnpm workspace interference: tools/pnpm-workspace.yaml only lists
   'skills-api'. pnpm traverses up from tools/claudeup-core and finds this
   config, skipping claudeup-core's own deps. Fix: --ignore-workspace.

2. pnpm-lock.yaml in tools/claudeup-core is missing vitest and
   @vitest/coverage-v8 entirely (they are in package.json devDependencies
   but were never committed to the lockfile). With --ignore-workspace pnpm
   reads the local frozen lockfile and never installs vitest →
   'sh: vitest: not found'. Fix: --no-frozen-lockfile so pnpm installs from
   package.json directly.

3. kanban version mismatch: plugins/kanban/plugin.json is 1.1.0 but
   .claude-plugin/marketplace.json still listed 1.0.0. The marketplace-sync
   integration test catches this. Fix: bump to 1.1.0 in marketplace.json.

Supersedes PR #6 (which fixed #1 and #3 but not #2)."
Copy link
Copy Markdown
Member Author

erudenko commented Apr 7, 2026

test


Generated by Claude Code

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.

1 participant