@raidiant/notifai 10.0.0 → 10.1.1

package/src/setup-destinations.test.ts

@@ -0,0 +1,41 @@
+import { describe, expect, it } from 'vitest'
+import {
+  companionPlatformLabel,
+  setupAccessUrl,
+  setupCompanionUrl,
+  supportPageUrl,
+} from './setup-destinations.js'
+
+describe('focused setup destinations', () => {
+  it('sends a hosted reader to the dashboard they can sign in to', () => {
+    expect(setupAccessUrl('https://api.notifai.sh')).toBe('https://app.notifai.sh/setup/access')
+    expect(setupCompanionUrl('https://notifai.fly.dev')).toBe(
+      'https://app.notifai.sh/setup/companion',
+    )
+  })
+
+  it('keeps a self-host on its own origin', () => {
+    expect(setupAccessUrl('https://notify.example.com/')).toBe(
+      'https://notify.example.com/setup/access',
+    )
+  })
+
+  it('carries the platform this terminal already asked about', () => {
+    expect(setupCompanionUrl('https://api.notifai.sh', 'iphone')).toBe(
+      'https://app.notifai.sh/setup/companion?platform=iphone',
+    )
+    expect(setupAccessUrl('https://api.notifai.sh', 'android')).toBe(
+      'https://app.notifai.sh/setup/access?platform=android',
+    )
+  })
+
+  it('keeps the omnibus help page separate from any setup errand', () => {
+    expect(supportPageUrl('https://api.notifai.sh')).toBe('https://app.notifai.sh/support')
+    expect(setupAccessUrl('https://api.notifai.sh')).not.toContain('/support')
+  })
+
+  it('names each platform the way a reader would', () => {
+    expect(companionPlatformLabel('iphone')).toBe('iPhone')
+    expect(companionPlatformLabel('android')).toBe('Android')
+  })
+})

package/src/setup-destinations.ts

@@ -0,0 +1,50 @@
+/**
+ * Where a cross-surface setup errand is finished.
+ *
+ * Every handoff opens the destination for exactly one errand, carrying the
+ * platform when this terminal already asked for it. The omnibus help page is
+ * help: it covers access, two install routes, billing, refunds, deletion,
+ * privacy and terms, and its own next step is to go somewhere else. Sending
+ * someone there for a named gate makes them find it themselves.
+ *
+ * The server prints these same paths into `next_action`, which is the value
+ * this CLI prefers whenever it has one. These builders are the fallback for
+ * the paths where it does not, and they must stay spelled the same.
+ */
+
+/** The two platforms that can receive a notification. Mac is not one. */
+export const COMPANION_PLATFORMS = ['iphone', 'android'] as const
+export type CompanionPlatform = (typeof COMPANION_PLATFORMS)[number]
+
+export function companionPlatformLabel(platform: CompanionPlatform): string {
+  return platform === 'iphone' ? 'iPhone' : 'Android'
+}
+
+/**
+ * The dashboard a reader can sign in to, which is not always the API origin.
+ * Only the hosted deployment splits them; a self-host serves both from one.
+ */
+function dashboardOrigin(baseUrl: string): string {
+  const normalized = baseUrl.replace(/\/+$/, '')
+  if (normalized === 'https://api.notifai.sh' || normalized === 'https://notifai.fly.dev') {
+    return 'https://app.notifai.sh'
+  }
+  return normalized
+}
+
+/** Getting this Account access to Notifai, and nothing else. */
+export function setupAccessUrl(baseUrl: string, platform?: CompanionPlatform): string {
+  const base = `${dashboardOrigin(baseUrl)}/setup/access`
+  return platform === undefined ? base : `${base}?platform=${platform}`
+}
+
+/** Installing the Companion App on one named platform. */
+export function setupCompanionUrl(baseUrl: string, platform?: CompanionPlatform): string {
+  const base = `${dashboardOrigin(baseUrl)}/setup/companion`
+  return platform === undefined ? base : `${base}?platform=${platform}`
+}
+
+/** General help: billing, policies, and everything setup is not. */
+export function supportPageUrl(baseUrl: string): string {
+  return `${dashboardOrigin(baseUrl)}/support`
+}

package/src/skill.test.ts

@@ -252,7 +252,7 @@ describe('Notifai agent skill', () => {
     // Harness names may appear only in setup/routing material, where the
     // harness IS the subject.
     for (const heading of ['## Decide whether to notify', '## Send', '## Ask a question', '## When the answer arrives']) {
-      expect(section(heading), heading).not.toMatch(/OpenCode|Orca|Cursor|Codex|Hermes/)
+      expect(section(heading), heading).not.toMatch(/OpenCode|OpenClaw|Orca|Cursor|Codex|Hermes/)
     }
   })
 
@@ -278,6 +278,17 @@ describe('Notifai agent skill', () => {
     expect(ask).toMatch(/tell me here/i)
   })
 
+  it('keeps local registration distinct from submission and delivery evidence', () => {
+    const ask = section('## Ask a question')
+    expect(ask).toMatch(/`registered: true`[\s\S]{0,180}local/i)
+    expect(ask).toMatch(/not yet been submitted as a\s+Notification\s+Request/i)
+    expect(ask).toMatch(/no Provider Acceptance/i)
+    expect(ask).toContain('notifai status <question_id> --json')
+    expect(ask).toMatch(/local.*frozen.*live.*answered.*withdrawn.*retired/is)
+    expect(ask).toMatch(/never call a question\s+sent\s+or\s+delivered\s+from registration alone/i)
+    expect(ask).toMatch(/inspect or close the original[\s\S]{0,160}do not register/i)
+  })
+
   it('requires an acknowledgement before any resumed work', () => {
     const answers = section('## When the answer arrives')
     const acknowledge = answers.indexOf('notifai acknowledge')
@@ -397,7 +408,7 @@ describe('Notifai agent skill', () => {
   it('moves per-harness mechanics into progressive disclosure', () => {
     expect(skill).toContain('[Harness setup and recovery](references/harness-setup.md)')
     expect(skill).toMatch(/read it when you are\s+installing hooks or diagnosing routing/i)
-    for (const harness of ['Claude Code', 'Codex', 'Cursor', 'OpenCode', 'Hermes']) {
+    for (const harness of ['Claude Code', 'Codex', 'Cursor', 'OpenCode', 'OpenClaw', 'Hermes']) {
       expect(harnessReference).toContain(`**${harness}:**`)
     }
     expect(harnessReference).toMatch(/exact `HERMES_SESSION_ID`/i)

package/src/ui/help.test.ts

@@ -0,0 +1,16 @@
+import { describe, expect, it } from 'vitest'
+import { rootHelpFooter } from './help.js'
+
+function stripAnsi(text: string): string {
+  return text.replace(/\u001B\[[0-9;]*m/g, '')
+}
+
+describe('rootHelpFooter', () => {
+  it('leads first-run with init, not the interactive app', () => {
+    const footer = stripAnsi(rootHelpFooter())
+    const start = footer.slice(footer.indexOf('Start here'))
+    const firstCommand = start.match(/notifai(?: init)?/)?.[0]
+    expect(firstCommand).toBe('notifai init')
+    expect(start.indexOf('notifai init')).toBeLessThan(start.search(/notifai\s{2,}/))
+  })
+})

package/src/ui/help.ts

@@ -1,8 +1,8 @@
 /**
  * Help that answers "what can I do here", not "what strings does argv accept".
  *
- * The old help listed seventeen commands in one flat column, with the two a new
- * user needs (`init`, `login`) sitting between a platform capability dump and
+ * The old help listed seventeen commands in one flat column, with the one a new
+ * user needs (`init`) sitting between a platform capability dump and
  * an entry point marked "Internal:". Everything was equally prominent, which is
  * the same as nothing being prominent.
  *
@@ -102,18 +102,17 @@ export const helpConfiguration: HelpConfiguration = {
 /**
  * Footer for the root help.
  *
- * Its whole job is to make the interactive app discoverable, because someone
- * reading a wall of flags has already demonstrated they did not know it was
- * there.
+ * First-run is `notifai init`. The interactive app is what a human opens
+ * later, once setup is no longer the thing in the way.
  */
 export function rootHelpFooter(): string {
   const lines = [
     '',
     style.heading('Start here'),
+    `  ${style.code('notifai init')}         ${style.dim(`${glyph.arrow} set this project up, step by step`)}`,
     `  ${style.code('notifai')}              ${style.dim(
-      `${glyph.arrow} the interactive app: status, settings, devices, test sends`,
+      `${glyph.arrow} after setup: status, settings, devices, test sends`,
     )}`,
-    `  ${style.code('notifai init')}         ${style.dim(`${glyph.arrow} set this project up, step by step`)}`,
     '',
     style.heading('First call, for an agent'),
     `  ${style.code('notifai send --kind done --title "All 42 tests passed" --body "Finished in 3m 10s. Next: review the release candidate."')}`,

package/src/commands-devices.test.ts

@@ -1,15 +0,0 @@
-import { describe, expect, it } from 'vitest'
-import { supportPageUrl } from './commands-devices.js'
-
-describe('supportPageUrl', () => {
-  it('routes hosted control-plane origins to the dashboard support surface', () => {
-    expect(supportPageUrl('https://api.notifai.sh')).toBe('https://app.notifai.sh/support')
-    expect(supportPageUrl('https://notifai.fly.dev/')).toBe('https://app.notifai.sh/support')
-  })
-
-  it('keeps a self-hosted deployment on its own support route', () => {
-    expect(supportPageUrl('https://notifai.example.test/')).toBe(
-      'https://notifai.example.test/support',
-    )
-  })
-})