amaprice 1.0.14 → 1.0.16

package/README.md

@@ -29,13 +29,13 @@ npx amaprice price "https://www.amazon.de/dp/B0DZ5P7JD6"
 ## Quickstart
 
 ```bash
-# one-shot lookup
+# one-shot lookup (no subscription)
 amaprice price "https://www.amazon.de/dp/B0DZ5P7JD6"
 
-# start tracking with a tier
+# start tracking + subscribe current user + auto-start background collector
 amaprice track B0DZ5P7JD6 --tier daily
 
-# subscribe current user to shared catalog product
+# or subscribe directly to an existing shared catalog product
 amaprice subscribe B0DZ5P7JD6
 
 # show history
@@ -43,6 +43,16 @@ amaprice history B0DZ5P7JD6 --limit 30
 
 # list tracked products
 amaprice list
+
+# list all subscriptions for current user (including paused)
+amaprice subscriptions --all
+
+# stop one product subscription for current user
+amaprice unsubscribe B0DZ5P7JD6
+
+# stop/start background collector service
+amaprice background off
+amaprice background on
 ```
 
 ## Input Modes
@@ -59,19 +69,47 @@ Short links from Amazon apps (for example `amzn.eu`, `amzn.to`, `a.co`) are acce
 | Command | Description |
 |---|---|
 | `amaprice [url\|asin]` | Shortcut for `amaprice price [url\|asin]` |
-| `amaprice price [url\|asin]` | One-shot lookup and silent history insert |
-| `amaprice track [url\|asin]` | Track product + current price (`--tier`, `--manual-tier`, `--auto-tier`, `--inactive`) |
+| `amaprice price [url\|asin]` | One-shot lookup (no subscription) and silent history insert |
+| `amaprice track [url\|asin]` | Track product + subscribe current user + auto-start background (`--tier`, `--manual-tier`, `--auto-tier`, `--inactive`) |
 | `amaprice subscribe [url\|asin]` | Subscribe current user to shared product catalog entry |
 | `amaprice unsubscribe <url\|asin>` | Disable current user subscription |
 | `amaprice subscriptions` | List user subscriptions with latest known prices |
 | `amaprice history <url\|asin>` | Show history (`--limit N`) |
-| `amaprice list` | List tracked products + latest price |
+| `amaprice list` | List current user subscriptions + latest price (default view) |
+| `amaprice list --global` | List global shared catalog tracked products |
 | `amaprice sync --limit <n>` | Run background sync for due products |
 | `amaprice background <on\|off\|status>` | Manage true background collector service |
 | `amaprice tier <url\|asin> <hourly\|daily\|weekly>` | Set tier/status (`--auto`, `--manual`, `--activate`, `--deactivate`) |
 
 All commands support `--json`.
 
+## Most Common User Flows
+
+### One-time Price Check (No Subscription)
+
+```bash
+amaprice price B0DZ5P7JD6
+```
+
+This returns the current price immediately and does not create a user subscription.
+
+### Subscribe + Run in Background
+
+```bash
+amaprice track B0DZ5P7JD6
+amaprice background status --json
+```
+
+`track` (and `subscribe`) auto-starts the background collector on macOS (`launchd`).
+You can close your terminal after this; the service keeps running.
+
+### Stop Product + Stop Background Service
+
+```bash
+amaprice unsubscribe B0DZ5P7JD6
+amaprice background off
+```
+
 ## Background Service (Auto)
 
 `track` and `subscribe` automatically ensure a true background collector service is running.
@@ -145,7 +183,11 @@ Run this SQL in Supabase SQL Editor:
 
 `supabase/migrations/20260220_add_price_history_currency.sql`
 
-These migrations add tier fields, indexes, telemetry, worker health rollups, and `price_history.currency`.
+`supabase/migrations/20260222_add_hybrid_orchestration.sql`
+
+`supabase/migrations/20260223_enforce_collector_first_claiming.sql`
+
+These migrations add tier fields, indexes, telemetry, worker health rollups, `price_history.currency`, collector orchestration tables/functions, and strict collector-first claim policy.
 
 Note: these files are additive migrations and expect existing `products` + `price_history` tables.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "amaprice",
-  "version": "1.0.14",
+  "version": "1.0.16",
   "description": "CLI tool to scrape and track Amazon product prices",
   "main": "src/scraper.js",
   "type": "commonjs",

package/src/background/service.js

@@ -114,15 +114,17 @@ function isLaunchdSupported(platform = process.platform) {
   return platform === 'darwin';
 }
 
-function getLaunchdDomain() {
+function getLaunchdDomains() {
   if (typeof process.getuid !== 'function') {
     throw new Error('launchd requires a POSIX uid');
   }
-  return `gui/${process.getuid()}`;
+  const uid = process.getuid();
+  return [`gui/${uid}`, `user/${uid}`];
 }
 
-function buildServiceTarget(label) {
-  return `${getLaunchdDomain()}/${label}`;
+function buildServiceTarget(label, domain = null) {
+  const safeDomain = domain || getLaunchdDomains()[0];
+  return `${safeDomain}/${label}`;
 }
 
 async function runLaunchctl(args, { allowFailure = false } = {}) {
@@ -233,24 +235,38 @@ async function getLaunchdServiceStatus({ label }) {
       backend: 'launchd',
       label,
       plistPath,
+      domain: null,
       installed: false,
       loaded: false,
       running: false,
     };
   }
 
-  const print = await runLaunchctl(['print', buildServiceTarget(label)], { allowFailure: true });
-  const output = `${print.stdout}\n${print.stderr}`;
-  const loaded = print.ok;
-  const running = loaded && (/state = running/i.test(output) || /pid = \d+/i.test(output));
+  const domains = getLaunchdDomains();
+  for (const domain of domains) {
+    const print = await runLaunchctl(['print', buildServiceTarget(label, domain)], { allowFailure: true });
+    if (!print.ok) continue;
+    const output = `${print.stdout}\n${print.stderr}`;
+    const running = /state = running/i.test(output) || /pid = \d+/i.test(output);
+    return {
+      backend: 'launchd',
+      label,
+      plistPath,
+      domain,
+      installed: true,
+      loaded: true,
+      running,
+    };
+  }
 
   return {
     backend: 'launchd',
     label,
     plistPath,
+    domain: domains[0] || null,
     installed: true,
-    loaded,
-    running,
+    loaded: false,
+    running: false,
   };
 }
 
@@ -263,7 +279,6 @@ async function enableLaunchdService({
   const plistPath = getLaunchdPlistPath(label);
   const logPath = getDaemonLogPath();
   const daemonEntry = getDaemonEntryPath();
-  const target = buildServiceTarget(label);
 
   await fs.mkdir(path.dirname(plistPath), { recursive: true });
   await fs.mkdir(path.dirname(logPath), { recursive: true });
@@ -284,33 +299,53 @@ async function enableLaunchdService({
   });
   await fs.writeFile(plistPath, plist, 'utf8');
 
-  // If service was previously disabled via `background off`, re-enable first.
-  await runLaunchctl(['enable', target], { allowFailure: true });
+  const domains = getLaunchdDomains();
+  const errors = [];
 
-  let bootstrap = await runLaunchctl(['bootstrap', getLaunchdDomain(), plistPath], { allowFailure: true });
-  if (!bootstrap.ok && !isAlreadyLoadedError(bootstrap)) {
-    // Recover from stale loaded/disabled state by clearing then bootstrapping once more.
-    await runLaunchctl(['bootout', target], { allowFailure: true });
-    await runLaunchctl(['enable', target], { allowFailure: true });
-    bootstrap = await runLaunchctl(['bootstrap', getLaunchdDomain(), plistPath], { allowFailure: true });
-  }
-  if (!bootstrap.ok && !isAlreadyLoadedError(bootstrap)) {
-    throw new Error(`Could not bootstrap launchd service: ${bootstrap.stderr || bootstrap.stdout || 'unknown error'}`);
-  }
+  for (const domain of domains) {
+    const domainTarget = buildServiceTarget(label, domain);
+
+    // Clean stale state first, then bootstrap fresh.
+    await runLaunchctl(['bootout', domainTarget], { allowFailure: true });
+    await runLaunchctl(['disable', domainTarget], { allowFailure: true });
+    await runLaunchctl(['enable', domainTarget], { allowFailure: true });
 
-  await runLaunchctl(['enable', target], { allowFailure: true });
-  const kick = await runLaunchctl(['kickstart', '-k', target], { allowFailure: true });
-  if (!kick.ok) {
-    await runLaunchctl(['start', label], { allowFailure: true });
+    let bootstrap = await runLaunchctl(['bootstrap', domain, plistPath], { allowFailure: true });
+    if (!bootstrap.ok && !isAlreadyLoadedError(bootstrap)) {
+      await runLaunchctl(['bootout', domainTarget], { allowFailure: true });
+      await runLaunchctl(['enable', domainTarget], { allowFailure: true });
+      bootstrap = await runLaunchctl(['bootstrap', domain, plistPath], { allowFailure: true });
+    }
+
+    if (!bootstrap.ok && !isAlreadyLoadedError(bootstrap)) {
+      errors.push(`${domain}: ${bootstrap.stderr || bootstrap.stdout || 'unknown error'}`);
+      continue;
+    }
+
+    await runLaunchctl(['enable', domainTarget], { allowFailure: true });
+    const kick = await runLaunchctl(['kickstart', '-k', domainTarget], { allowFailure: true });
+    if (!kick.ok) {
+      await runLaunchctl(['start', label], { allowFailure: true });
+    }
+
+    const status = await getLaunchdServiceStatus({ label });
+    if (status.loaded) {
+      return status;
+    }
+    errors.push(`${domain}: bootstrapped but service not loaded`);
   }
 
-  return getLaunchdServiceStatus({ label });
+  const msg = errors.length > 0 ? errors.join(' | ') : 'unknown error';
+  throw new Error(`Could not bootstrap launchd service: ${msg}`);
 }
 
 async function disableLaunchdService({ label }) {
   const plistPath = getLaunchdPlistPath(label);
-  await runLaunchctl(['bootout', buildServiceTarget(label)], { allowFailure: true });
-  await runLaunchctl(['disable', buildServiceTarget(label)], { allowFailure: true });
+  for (const domain of getLaunchdDomains()) {
+    const target = buildServiceTarget(label, domain);
+    await runLaunchctl(['bootout', target], { allowFailure: true });
+    await runLaunchctl(['disable', target], { allowFailure: true });
+  }
   try {
     await fs.unlink(plistPath);
   } catch (err) {
@@ -342,7 +377,7 @@ async function ensureBackgroundOn({
   const { collector, statePath } = await ensureCollectorEnabled({
     userId,
     collectorName,
-    status: 'active',
+    status: 'paused',
   });
 
   const service = await enableLaunchdService({
@@ -352,6 +387,10 @@ async function ensureBackgroundOn({
     userId,
   });
 
+  if (!service.loaded) {
+    throw new Error('launchd service did not load');
+  }
+
   await heartbeatCollector({
     collectorId: collector.id,
     status: 'active',
@@ -510,6 +549,8 @@ module.exports.__test = {
   resolvePollSeconds,
   getLaunchdLabel,
   getLaunchdPlistPath,
+  getLaunchdDomains,
+  buildServiceTarget,
   renderLaunchdPlist,
   isLaunchdSupported,
 };