Skip to content

feat: defer mobile (isDesktopOnly: true) + honest docs; release 0.1.6#64

Merged
vreshch merged 1 commit into
masterfrom
feature/desktop-only
Jun 21, 2026
Merged

feat: defer mobile (isDesktopOnly: true) + honest docs; release 0.1.6#64
vreshch merged 1 commit into
masterfrom
feature/desktop-only

Conversation

@vreshch

@vreshch vreshch commented Jun 21, 2026

Copy link
Copy Markdown
Member

Makes the plugin honestly desktop-only instead of claiming mobile support that doesn't complete.

Why: mobile sign-in doesn't round-trip inside Obsidian's Capacitor WebView — window.open(_blank) opens an in-app browser (no shared auth session) and the obsidian:// deep-link callback doesn't reliably return, so users get stuck signed-out and every action is gated off. The git/VaultFs layer itself is mobile-safe; sign-in + first-sync .git detection are the unverified parts.

Change

  • manifest.json: isDesktopOnly: false → true — Obsidian now hides/disables it on phones (no broken install), instead of offering a plugin that can't sign in.
  • README: status banner + Features + Getting started reworded to desktop-only; "Mobile support (experimental)" → "Mobile (planned)" (groundwork in place, pending device-verified sign-in + sync).
  • CLAUDE.md + a main.ts comment: corrected the stale "VaultFs built, not yet wired / desktop uses node fs" claims (VaultFs is the git fs on every platform; node fs is only the desktop ~/.agentage mirror).
  • Mobile code (VaultFs, ribbon/modal, deep-link handler) stays in the repo for re-enablement.

Release: bumps 0.1.5 → 0.1.6. npm run verify green (66 tests, build, host/docs/bundle checks).

Re-enablement playbook (sign-in fix + mobile smoke) tracked in the submission/feature doc.

@github-actions

Copy link
Copy Markdown

🎉 PR Validation ✅ PASSED

Commit: 3b0c8589bf1ddfdf928171cde01b39807bf2c9f3
Branch: feature/desktop-only

Checks:

  • ✅ Dependencies installed
  • ✅ Type check passed
  • ✅ Linting passed
  • ✅ Format check passed
  • ✅ Tests + coverage passed
  • ✅ Build successful
  • ✅ Store-compliance checks passed
  • ✅ Bundle is mobile-safe

Ready to merge!


🔗 View workflow run

@vreshch vreshch merged commit bad2eca into master Jun 21, 2026
1 check passed
@vreshch vreshch deleted the feature/desktop-only branch June 21, 2026 10:10
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