registry / thinkpool-pair / 0.7.145

thinkpool-pair@0.7.145

Share a local coding-agent CLI (Claude Code, Codex, Gemini, Aider, …) into a ThinkPool Code room, live.

AI Security Review

scanned 7d ago · by lpm-firewall-ai

No confirmed malicious attack surface. The package is a user-invoked bridge for sharing local coding-agent CLIs into ThinkPool rooms, with expected network, subprocess, local state, and optional service persistence capabilities.

Static reason

One or more suspicious static signals were detected.; previous stored version diff introduced dangerous source

Trigger

User runs thinkpool-pair, login, provider, install-service, or interacts with a paired ThinkPool room.

Impact

Runtime can relay terminal/agent input and output, download room files, upload mockup/image artifacts, and optionally install a persistent service when explicitly requested.

Mechanism

Package-aligned remote coding-agent bridge with permission-gated agent controls.

Rationale

Static inspection found powerful but product-aligned functionality for a remote coding-agent bridge, not install-time execution, hidden exfiltration, or unconsented AI-agent control-surface mutation. The scanner's critical secret is a public Supabase anon key and persistence is explicit install-service behavior.

Evidence

package.jsonbridge.mjsservice.mjsaccount.mjsauth-store.mjsprovider.mjsclaude-session.mjs~/.thinkpool-pair/*~/Library/LaunchAgents/io.thinkpool.pair.*.plist~/.config/systemd/user/io.thinkpool.pair.*.service%APPDATA%/Microsoft/Windows/Start Menu/Programs/Startup/thinkpool-pair-*.cmd

Network endpoints3

daytvtakmlixpfbbqzjd.supabase.cothinkpool.ioregistry.npmjs.org/thinkpool-pair/latest

Decision evidence

public snapshot

AI called this Clean at 86.0% confidence as Benign with medium false-positive risk.

Evidence for block

bridge.mjs exposes user-invoked remote coding-agent control over Supabase room channels.
service.mjs can install boot-persistent launchd/systemd/Startup service, but only via install-service command.
bridge.mjs downloads room-supplied file URLs to temp dir during runtime file-put events.
claude-session.mjs supports bypassPermissions and agent lane spawning when chosen/approved in room.

Evidence against

package.json has no install/preinstall/postinstall lifecycle hooks; bin is bridge.mjs only.
Embedded Supabase JWT in bridge.mjs is documented anon public client key, not a private secret.
bridge.mjs validates room codes before filesystem/service label use and gates service install behind explicit CLI subcommands.
claude-session.mjs implements PreToolUse risk classification, permission cards, deny-fail behavior, and caps/kill switches for cross-terminal tools.
account.mjs/auth-store.mjs store only ThinkPool login tokens locally under ~/.thinkpool-pair after explicit login.
Network hosts are package-aligned: ThinkPool web, Supabase backend, npm registry update checks.

Behavioral surface

Source

ChildProcessCryptoEnvironmentVarsFilesystemNetworkShell

Supply chain

HighEntropyStringsUrlStrings

Manifest

NoLicense

scanned 27 file(s), 555 KB of source, external domains: api.anthropic.com, api.z.ai, daytvtakmlixpfbbqzjd.supabase.co, openrouter.ai, registry.npmjs.org, thinkpool.io, www.apple.com

Source & flagged code

4 flagged

bridge.mjsView file

Lines 74-114javascript

74// a flow-revert kill and the next dispatch wave loses the pending resume record, so the task

75// re-dispatches COLD (fresh lane, no resume). That's safe: the healed transcript survives on

76// disk (sanitizeSession wrote it), so a cold re-dispatch can't 400 on a dangling tool_use — it

77// just loses the resume/revert-target carry. Not worth persisting for that narrow window.

78const flowRedispatch = new Map()

79// FL-B2 — per-flow HARD budget ledger (flowId → budget). Accumulates each lane's

80// completed-turn output tokens (onEvent 'result') so the autopilot cap stops the next

81// wave BEFORE overrun. Lives bridge-side because waves dispatch across separate

82// broadcasts; without persistent state the cap can never bite.

83const flowBudgets = new Map()

84import { formatPeek, PEEK, siblingsOf, resolveSibling, crossPostDecision, CROSSPOST, spawnDecision, CROSSROOM, formatPairRoster, crossRoomPostDecision, formatRoomNow } from './cross-terminal.mjs'

85import { turnInFlight } from './update-gate.mjs'

86import { saveSession, flushSession, deleteSession, loadAll, canResume, loadPtyId, savePtyId, loadNames, saveNames, appendDurableEvents, seedDurableEvents, readDurablePage, readDurableOldestSeq, loadSummary, saveSummary } from './session-store.mjs'

87import { stampEvent, makeSeqCounter, maxSeq, seqable, capReplayEvents, chunkReplayEvents, boundEventForBroadcast, inlineImageBlocks } from './event-id.mjs'

88import { makeThrottledTrack } from './presence.mjs'

90// Public client creds (the same anon values the web app ships — safe to embed).

91// Override with TP_SUPABASE_URL / TP_SUPABASE_ANON if you ever need to.

92const SUPABASE_URL = process.env.TP_SUPABASE_URL || 'https://daytvtakmlixpfbbqzjd.supabase.co'

93const WEB_BASE = process.env.TP_WEB_BASE || 'https://thinkpool.io'

94const SUPABASE_ANON = process.env.TP_SUPABASE_ANON || 'eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6ImRheXR2dGFrbWxpeHBmYmJxempkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3NzgwNTMxNzEsImV4cCI6MjA5MzYyOTE3MX0.zYmz8bHUNEY4STUr2JuXChAsKwxNwkwFHe1Hd0Betqo'

Critical

Critical Secret

Package contains a critical-looking secret pattern.

bridge.mjsView on unpkg · L94

Critical

Secret Pattern

Supabase service role key (JWT) in bridge.mjs

bridge.mjsView on unpkg · L94

96// Owner access token for AUTHED web writes (currently /api/code-mockup, whip L16).

97// The room-bridge is anon by design; the account supervisor (account.mjs) owns the

98// session, so it hands a fresh owner token in at spawn (TP_ACCESS_TOKEN) and pushes

99// a rotated one over IPC ({ t:'token' }) on each refresh. Stays null for a

100// directly-started bridge with no account login — those writes go unattributed,

101// never owner-spoofed (the server treats a missing/!owner token accordingly).

102let codeAuthToken = process.env.TP_ACCESS_TOKEN || null

103

104// Per-terminal rolling scrollback (raw chars). Replayed to joining clients.

105// Kept comfortably under realtime payload limits once base64'd.

106const SCROLLBACK_MAX = 120_000

107

108// node-pty is a native module — loaded lazily so a friendly message shows

109// if it isn't built yet.

110// Structured Claude (the default) runs through the Agent SDK and needs NO PTY,

111// so a failed node-pty build must not block the bridge — it only disables the

112// raw-PTY path (other CLIs / TP_PTY=1), which we flag clearly if it's used.

113let pty = null

114try { pty = (await import('node-pty')).default ?? (await import('node-pty')) }

•matchType = previous_version_dangerous_delta matchedPackage = thinkpool-pair@0.7.144 matchedIdentity = npm:dGhpbmtwb29sLXBhaXI:0.7.144 similarity = 0.926 summary = stored previous version shares package body but lacks this dangerous source file

High

Previous Version Dangerous Delta

This package version adds a dangerous source file absent from the previous stored version; route for source-aware review.

bridge.mjsView on unpkg

service.mjsView file

Lines 2-42javascript

2 service.mjs — install the bridge as a boot-persistent OS service

3 so a customer's bridge survives reboot/logout and auto-restarts

4 on crash, on every platform:

5 • macOS → launchd LaunchAgent (KeepAlive)

6 • Linux → systemd --user unit (Restart=always)

7 • Windows → Startup-folder .cmd running

Source & flagged code

4 flagged · loading source

bridge.mjsView file

94patternName = supabase_service_key severity = critical line = 94 matchedText = const SU...tqo'

Critical

Critical Secret

Package contains a critical-looking secret pattern.

bridge.mjsView on unpkg · L94

94patternName = supabase_service_key severity = critical line = 94 matchedText = const SU...tqo'

Critical

Secret Pattern

Supabase service role key (JWT) in bridge.mjs

bridge.mjsView on unpkg · L94

High

Previous Version Dangerous Delta

This package version adds a dangerous source file absent from the previous stored version; route for source-aware review.

bridge.mjsView on unpkg

service.mjsView file

22import path from 'node:path' L23: import { execSync } from 'node:child_process' L24: ... L26: // exactly one account service per machine (a second install replaces it). L27: // Room codes flow UNESCAPED into launchctl/systemd shell strings + plist paths, so L28: // this is the security chokepoint: reject anything that isn't a plain alnum code ... L44: // `home` is the platform-correct, runtime-expanded home token ("$HOME" on darwin under L45: // /bin/bash; "$$HOME" inside a systemd unit so systemd emits a literal $HOME to bash). L46: const cacheWipe = (home) => `rm -rf ${home}/.npm/_npx/*/node_modules/thinkpool-pair 2>/dev/null` ... L48: // The version installing the service. By DEFAULT the service is pinned to this exact L49: // version (not @latest) so a future bad npm publish can't auto-roll to every machine's L50: // always-on bridge — that's what spread the broken 0.7.49 on 2026-06-22. Updates become

Medium

Install Persistence

Source writes installer persistence such as shell profile or service configuration.

service.mjsView on unpkg · L22

Findings

2 Critical1 High3 Medium5 Low

CriticalCritical Secretbridge.mjs

CriticalSecret Patternbridge.mjs

HighPrevious Version Dangerous Deltabridge.mjs

MediumNetwork

MediumEnvironment Vars

MediumInstall Persistenceservice.mjs

LowScripts Present

LowFilesystem

LowHigh Entropy Strings

LowUrl Strings

LowNo License

Source & flagged code

4 flagged

bridge.mjsView file

Lines 74-114javascript

74// a flow-revert kill and the next dispatch wave loses the pending resume record, so the task

75// re-dispatches COLD (fresh lane, no resume). That's safe: the healed transcript survives on

76// disk (sanitizeSession wrote it), so a cold re-dispatch can't 400 on a dangling tool_use — it

77// just loses the resume/revert-target carry. Not worth persisting for that narrow window.

78const flowRedispatch = new Map()

79// FL-B2 — per-flow HARD budget ledger (flowId → budget). Accumulates each lane's

80// completed-turn output tokens (onEvent 'result') so the autopilot cap stops the next

81// wave BEFORE overrun. Lives bridge-side because waves dispatch across separate

82// broadcasts; without persistent state the cap can never bite.

83const flowBudgets = new Map()

84import { formatPeek, PEEK, siblingsOf, resolveSibling, crossPostDecision, CROSSPOST, spawnDecision, CROSSROOM, formatPairRoster, crossRoomPostDecision, formatRoomNow } from './cross-terminal.mjs'

85import { turnInFlight } from './update-gate.mjs'

87import { stampEvent, makeSeqCounter, maxSeq, seqable, capReplayEvents, chunkReplayEvents, boundEventForBroadcast, inlineImageBlocks } from './event-id.mjs'

88import { makeThrottledTrack } from './presence.mjs'

90// Public client creds (the same anon values the web app ships — safe to embed).

91// Override with TP_SUPABASE_URL / TP_SUPABASE_ANON if you ever need to.

92const SUPABASE_URL = process.env.TP_SUPABASE_URL || 'https://daytvtakmlixpfbbqzjd.supabase.co'

93const WEB_BASE = process.env.TP_WEB_BASE || 'https://thinkpool.io'

Critical

Critical Secret

Package contains a critical-looking secret pattern.

bridge.mjsView on unpkg · L94

Critical

Secret Pattern

Supabase service role key (JWT) in bridge.mjs

bridge.mjsView on unpkg · L94

96// Owner access token for AUTHED web writes (currently /api/code-mockup, whip L16).

97// The room-bridge is anon by design; the account supervisor (account.mjs) owns the

98// session, so it hands a fresh owner token in at spawn (TP_ACCESS_TOKEN) and pushes

99// a rotated one over IPC ({ t:'token' }) on each refresh. Stays null for a

100// directly-started bridge with no account login — those writes go unattributed,

101// never owner-spoofed (the server treats a missing/!owner token accordingly).

102let codeAuthToken = process.env.TP_ACCESS_TOKEN || null

103

104// Per-terminal rolling scrollback (raw chars). Replayed to joining clients.

105// Kept comfortably under realtime payload limits once base64'd.

106const SCROLLBACK_MAX = 120_000

107

108// node-pty is a native module — loaded lazily so a friendly message shows

109// if it isn't built yet.

110// Structured Claude (the default) runs through the Agent SDK and needs NO PTY,

111// so a failed node-pty build must not block the bridge — it only disables the

112// raw-PTY path (other CLIs / TP_PTY=1), which we flag clearly if it's used.

113let pty = null

114try { pty = (await import('node-pty')).default ?? (await import('node-pty')) }

High

Previous Version Dangerous Delta

This package version adds a dangerous source file absent from the previous stored version; route for source-aware review.

bridge.mjsView on unpkg

service.mjsView file

Lines 2-42javascript

2 service.mjs — install the bridge as a boot-persistent OS service

3 so a customer's bridge survives reboot/logout and auto-restarts

4 on crash, on every platform:

5 • macOS → launchd LaunchAgent (KeepAlive)

6 • Linux → systemd --user unit (Restart=always)

7 • Windows → Startup-folder .cmd running