@goliapkg/sentori-cli 0.5.3 → 1.0.0

package/package.json

@@ -1,13 +1,14 @@
 {
   "name": "@goliapkg/sentori-cli",
-  "version": "0.5.3",
-  "description": "Sentori CLI — upload source maps + other release artifacts to a Sentori server. Thin wrapper that downloads the matching prebuilt Rust binary at install time.",
-  "license": "MIT",
+  "version": "1.0.0",
+  "description": "Sentori CLI — upload source maps so dashboard stack traces resolve to your original source.",
+  "license": "Apache-2.0 OR MIT",
+  "author": "GOLIA K.K. <takagi@golia.jp> (https://golia.jp)",
   "homepage": "https://sentori.golia.jp",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/goliajp/sentori.git",
-    "directory": "cli/npm"
+    "directory": "sdk/cli"
   },
   "bugs": {
     "url": "https://github.com/goliajp/sentori/issues"
@@ -15,25 +16,34 @@
   "keywords": [
     "sentori",
     "cli",
-    "sourcemap",
+    "sourcemaps",
     "error-tracking"
   ],
+  "type": "module",
   "bin": {
-    "sentori-cli": "bin/sentori-cli.js"
-  },
-  "scripts": {
-    "postinstall": "node scripts/postinstall.js || echo 'sentori-cli postinstall failed — run `npx @goliapkg/sentori-cli` once you have network'"
+    "sentori-cli": "./lib/index.js"
   },
+  "main": "./lib/index.js",
+  "types": "./lib/index.d.ts",
   "files": [
-    "bin/",
-    "scripts/",
+    "lib/",
+    "src/",
     "README.md"
   ],
   "engines": {
     "node": ">=18"
   },
-  "os": ["darwin", "linux"],
-  "cpu": ["x64", "arm64"],
+  "scripts": {
+    "build": "tsc -p tsconfig.json",
+    "typecheck": "tsc --noEmit",
+    "test": "bun test",
+    "prepack": "bun run build"
+  },
+  "devDependencies": {
+    "@types/bun": "latest",
+    "@types/node": "*",
+    "typescript": "^7"
+  },
   "publishConfig": {
     "access": "public"
   }

package/src/__tests__/lenient.test.ts

@@ -0,0 +1,32 @@
+// Build-time iron rule: upload failure exits 0 with a friendly,
+// actionable note; --strict opts into a real non-zero.
+
+import { describe, expect, test } from 'bun:test'
+
+import { isStrict, lenientFail, stripStrict } from '../lenient.js'
+
+describe('lenient upload contract', () => {
+  test('default: failure exits 0', () => {
+    const code = lenientFail(false, {
+      failure: 'network down',
+      impact: 'stacks stay minified',
+      retry: 'sentori-cli upload sourcemap ...',
+    })
+    expect(code).toBe(0)
+  })
+
+  test('--strict: failure exits 1', () => {
+    const code = lenientFail(true, {
+      failure: 'network down',
+      impact: 'stacks stay minified',
+      retry: 'retry-cmd',
+    })
+    expect(code).toBe(1)
+  })
+
+  test('strict flag parses and strips', () => {
+    expect(isStrict(['--release', 'r', '--strict'])).toBe(true)
+    expect(isStrict(['--release', 'r'])).toBe(false)
+    expect(stripStrict(['--strict', 'a'])).toEqual(['a'])
+  })
+})

package/src/__tests__/mcp.test.ts

@@ -0,0 +1,32 @@
+// The MCP tool surface mirrors the /api closed loop 1:1.
+
+import { describe, expect, test } from 'bun:test'
+
+import { buildTools } from '../mcp.js'
+
+describe('mcp tools', () => {
+  const tools = buildTools()
+
+  test('exactly the four closed-loop tools exist', () => {
+    expect(tools.map((t) => t.name).sort()).toEqual([
+      'sentori_issue_bundle',
+      'sentori_issue_list',
+      'sentori_issue_note',
+      'sentori_issue_resolve',
+    ])
+  })
+
+  test('every tool has a description and an object schema', () => {
+    for (const t of tools) {
+      expect(t.description.length).toBeGreaterThan(20)
+      expect((t.inputSchema as { type?: string }).type).toBe('object')
+    }
+  })
+
+  test('bundle/note/resolve require issueId', () => {
+    for (const name of ['sentori_issue_bundle', 'sentori_issue_note', 'sentori_issue_resolve']) {
+      const t = tools.find((x) => x.name === name)!
+      expect((t.inputSchema as { required?: string[] }).required).toContain('issueId')
+    }
+  })
+})

package/src/__tests__/native-artifacts.test.ts

@@ -0,0 +1,125 @@
+import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+
+import {
+  dsymSlicesFromDwarfdump,
+  dwarfBinariesIn,
+  uploadDsym,
+  uploadMapping,
+} from '../native-artifacts.js'
+
+const ADMIN = {
+  apiUrl: 'https://api.example.com/',
+    token: 'sk_test',
+}
+
+let dir = ''
+const origFetch = globalThis.fetch
+let calls: { body: Buffer | null; headers: Headers; method: string; url: string }[]
+beforeEach(async () => {
+  dir = await mkdtemp(join(tmpdir(), 'sentori-cli-native-'))
+  calls = []
+})
+afterEach(async () => {
+  globalThis.fetch = origFetch
+  await rm(dir, { force: true, recursive: true })
+})
+
+function mockFetch(status = 200, respBody = ''): void {
+  globalThis.fetch = (async (url: Request | string | URL, init?: RequestInit) => {
+    const body = init?.body
+    calls.push({
+      body: body instanceof Uint8Array ? Buffer.from(body) : null,
+      headers: new Headers(init?.headers),
+      method: init?.method ?? 'GET',
+      url: String(url),
+    })
+    return new Response(respBody, { status })
+  }) as typeof fetch
+}
+
+describe('uploadMapping', () => {
+  test('POSTs raw bytes to /mappings with Bearer + release query', async () => {
+    mockFetch()
+    const path = join(dir, 'mapping.txt')
+    await writeFile(path, '# pg_map_id: abc\nfoo.bar -> a.b:\n')
+    await uploadMapping({ ...ADMIN, path, release: 'app@1.0.0+1' })
+    expect(calls).toHaveLength(1)
+    expect(calls[0]?.method).toBe('POST')
+    expect(calls[0]?.url).toBe(
+      'https://api.example.com/v1/releases/app%401.0.0%2B1/artifacts',
+    )
+    expect(calls[0]?.headers.get('authorization')).toBe('Bearer sk_test')
+  })
+
+  test('sets x-sentori-debug-id when given', async () => {
+    mockFetch()
+    const path = join(dir, 'mapping.txt')
+    await writeFile(path, 'x')
+    await uploadMapping({ ...ADMIN, debugId: '1234-uuid', path, release: 'r' })
+    expect(calls[0]?.url).toContain('/v1/releases/r/artifacts')
+  })
+
+  test('throws on empty file', async () => {
+    const path = join(dir, 'empty.txt')
+    await writeFile(path, '')
+    await expect(uploadMapping({ ...ADMIN, path })).rejects.toThrow('empty mapping file')
+  })
+
+  test('throws on non-2xx with server detail', async () => {
+    mockFetch(403, 'forbidden')
+    const path = join(dir, 'mapping.txt')
+    await writeFile(path, 'x')
+    await expect(uploadMapping({ ...ADMIN, path })).rejects.toThrow('403')
+  })
+})
+
+describe('uploadDsym (explicit single-slice)', () => {
+  test('POSTs to /dsyms with x-sentori-debug-id, x-sentori-arch, release + objectName', async () => {
+    mockFetch()
+    const path = join(dir, 'Foo.dSYM/Contents/Resources/DWARF/Foo')
+    await mkdir(join(dir, 'Foo.dSYM/Contents/Resources/DWARF'), { recursive: true })
+    await writeFile(path, 'macho-bytes')
+    const r = await uploadDsym({
+      ...ADMIN,
+      arch: 'arm64',
+      debugId: '1234abcd-1234-1234-1234-1234567890ab',
+      objectName: 'Foo',
+      path,
+      release: 'app@1.0.0+1',
+    })
+    expect(calls).toHaveLength(1)
+    expect(calls[0]?.method).toBe('POST')
+    expect(calls[0]?.url).toContain('/v1/releases/app%401.0.0%2B1/artifacts')
+    expect(r.slices).toEqual([{ arch: 'arm64', debugId: '1234ABCD-1234-1234-1234-1234567890AB' }])
+  })
+})
+
+describe('helpers', () => {
+  test('dwarfBinariesIn walks a .dSYM bundle', async () => {
+    const bundle = join(dir, 'Foo.dSYM')
+    const dwarfDir = join(bundle, 'Contents/Resources/DWARF')
+    await mkdir(dwarfDir, { recursive: true })
+    await writeFile(join(dwarfDir, 'Foo'), 'x')
+    await writeFile(join(dwarfDir, '.DS_Store'), 'noise')
+    const found = dwarfBinariesIn(bundle)
+    expect(found.map((p) => p.replace(bundle + '/', ''))).toEqual([
+      'Contents/Resources/DWARF/Foo',
+    ])
+  })
+
+  test('dwarfBinariesIn on a plain file returns just that file', async () => {
+    const f = join(dir, 'binary')
+    await writeFile(f, 'x')
+    expect(dwarfBinariesIn(f)).toEqual([f])
+  })
+
+  test('dsymSlicesFromDwarfdump returns [] when dwarfdump fails (or is absent)', () => {
+    // pass a path that doesn't exist; dwarfdump exits non-zero (or isn't installed)
+    const slices = dsymSlicesFromDwarfdump(join(dir, 'nope.dSYM'))
+    expect(slices).toEqual([])
+  })
+})

package/src/__tests__/probes.test.ts

@@ -0,0 +1,41 @@
+// The probe scanner: finds sentori.probe('REF') call sites, skips
+// junk dirs, dedupes.
+
+import { mkdtemp, mkdir, rm, writeFile } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+
+import { scanProbes } from '../probes.js'
+
+let dir = ''
+beforeEach(async () => {
+  dir = await mkdtemp(join(tmpdir(), 'sentori-probes-'))
+})
+afterEach(async () => {
+  await rm(dir, { force: true, recursive: true })
+})
+
+describe('scanProbes', () => {
+  test('finds refs across quote styles and dedupes', async () => {
+    await writeFile(
+      join(dir, 'a.ts'),
+      `sentori.probe('SENT-1'); probe("SENT-2", {x:1}); sentori.probe(\`SENT-1\`)`,
+    )
+    await writeFile(join(dir, 'b.tsx'), `if (bad) { sentori.probe('SENT-3') }`)
+    expect(scanProbes(dir)).toEqual(['SENT-1', 'SENT-2', 'SENT-3'])
+  })
+
+  test('skips node_modules and non-source files', async () => {
+    await mkdir(join(dir, 'node_modules'), { recursive: true })
+    await writeFile(join(dir, 'node_modules', 'x.ts'), `probe('NOPE')`)
+    await writeFile(join(dir, 'notes.md'), `probe('ALSO-NOPE')`)
+    expect(scanProbes(dir)).toEqual([])
+  })
+
+  test('dynamic refs are invisible (documented limitation)', async () => {
+    await writeFile(join(dir, 'c.ts'), 'sentori.probe(ref)')
+    expect(scanProbes(dir)).toEqual([])
+  })
+})

package/src/__tests__/react-native.test.ts

@@ -0,0 +1,37 @@
+import { mkdtemp, rm, writeFile } from 'node:fs/promises'
+import { tmpdir } from 'node:os'
+import { join } from 'node:path'
+
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test'
+
+import { composeSourceMaps, resolveComposeScript } from '../react-native.js'
+
+let dir = ''
+beforeEach(async () => {
+  dir = await mkdtemp(join(tmpdir(), 'sentori-cli-rn-'))
+})
+afterEach(async () => {
+  await rm(dir, { force: true, recursive: true })
+})
+
+describe('react-native helpers', () => {
+  test('resolveComposeScript returns null when react-native is not installed', () => {
+    // This package doesn't depend on react-native, so from sdk/cli's
+    // own node_modules there's nothing to find.
+    expect(resolveComposeScript()).toBeNull()
+  })
+
+  test('composeSourceMaps throws "no such file" for a missing input', () => {
+    expect(() => composeSourceMaps(join(dir, 'nope.map'), join(dir, 'nope2.map'))).toThrow(
+      'no such file',
+    )
+  })
+
+  test('composeSourceMaps throws a helpful message when react-native is absent', async () => {
+    await writeFile(join(dir, 'a.packager.map'), '{"version":3}')
+    await writeFile(join(dir, 'a.hbc.map'), '{"version":3}')
+    expect(() =>
+      composeSourceMaps(join(dir, 'a.packager.map'), join(dir, 'a.hbc.map')),
+    ).toThrow(/compose-source-maps\.js/)
+  })
+})