@ngockhoale/ukit@3.4.31
Install/update an index-first AI workspace for Claude Code, OpenAI Codex and omp (Oh My Pi).
AI Security Review
scanned 12d ago · by lpm-firewall-aiStatic inspection found conditional credential exposure: the decision URL is independently configurable, while key resolution can fall back to the shared Anthropic token, and no destination allowlist is enforced. The decision stage gates use of this client, and the default endpoint is package-aligned, so the finding warrants a warning rather than a block.
Static reason
No blocking static signals were detected.
Trigger
A decision-plane caller runs with a non-off stage and a configured UKIT_DECISION_BASE_URL or saved decision gateway URL, while no separate decision API key overrides the shared token.
Impact
A configured endpoint outside the intended service can receive the shared token. No evidence establishes that this happens by default or that the default endpoint is malicious.
Mechanism
The client resolves the decision URL and API key independently, then sends the key in an authorization header with a POST request to the resolved URL.
AI rationale
Static inspection found conditional credential exposure: the decision URL is independently configurable, while key resolution can fall back to the shared Anthropic token, and no destination allowlist is enforced. The decision stage gates use of this client, and the default endpoint is package-aligned, so the finding warrants a warning rather than a block.
Evidence
src/core/gatewayProbe.jssrc/decision/client.jssrc/core/agentRuntime/vmEngine.js
Network endpoints1
openai.unicjsc.com
Decision evidence
public snapshotAI called this Suspicious at 89.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- src/core/gatewayProbe.js accepts a configurable decision URL and can fall back to a shared UNIC token; src/decision/client.js sends the resolved authorization header to that URL.
- The decision client is reachable when the VM decision stage is enabled; package.json exposes a CLI and declares no install lifecycle hook.
- The required agent-related files show benchmark and probe code, guarded workspace maintenance, hook repair, and a watchdog delegating to its coordinator; no separate concrete malicious behavior was established.
Evidence against
- The decision request uses the package's intended UNIC decision service by default, and runtime calls are stage-gated; the exposure requires a configured alternate endpoint.
Behavioral surface
ChildProcessCryptoDynamicRequireEnvironmentVarsFilesystemShell
HighEntropyStringsUrlStrings
Source & flagged code
6 flagged · loading sourcescripts/bench/runGold.mjsView file
121const rootDir = projectRoot ?? REPO_ROOT;
L122: const { compileContext } = await import(path.join(REPO_ROOT, 'src/core/codeintel/compiler.js'));
L123: const { routeTask } = await import(path.join(REPO_ROOT, 'src/core/codeintel/router.js'));
Medium
Dynamic Require
Package source references dynamic require/import behavior.
scripts/bench/runGold.mjsView on unpkg · L121scripts/release/verify-release.mjsView file
1import { spawn } from 'node:child_process';
L2: import { readFileSync, writeFileSync, unlinkSync, existsSync } from 'node:fs';
...
L6:
L7: const rootDir = process.cwd();
L8: const npmCacheDir = path.join(os.tmpdir(), 'ukit-npm-cache');
...
L14: // UKIT_VERIFY_STEP_TIMEOUT_MS overrides the step budget for tests.
L15: const STEP_TIMEOUT_MS = Number(process.env.UKIT_VERIFY_STEP_TIMEOUT_MS) || 10 * 60 * 1000; // 10 min
L16: const REGISTRY_TIMEOUT_MS = Number(process.env.UKIT_VERIFY_REGISTRY_TIMEOUT_MS) || 60 * 1000; // 1 min
...
L18: // Opt-in post-publish registry-parity check (Release Policy invariant: npm and git always hold
L19: // the same latest version). Only meaningful AFTER `npm publish` — before publish the registry is
L20: // behind by definition — so it is gated behind --post-
Low
Weak Crypto
Package source references weak cryptographic algorithms.
Findings
2 High5 Medium5 Low
HighPayload In Excluded Dirtemplate_project/.claude/hooks/task-watchdog.sh
HighSemantic Analysis Limitedscripts/audit/decision-coverage.mjs#virtual:normalized:round1
MediumDynamic Requirescripts/bench/runGold.mjs
MediumEnvironment Vars
MediumShips Build Helpertemplate_project/.claude/hooks/task-watchdog.sh
MediumStructural Risk Force Deep Review
MediumSuspicious Lifecycle Evidencepackage.json
LowScripts Present
LowWeak Cryptoscripts/release/verify-release.mjs
LowFilesystem
LowHigh Entropy Strings
LowUrl Strings
Affected versions and remediation
This report applies to @ngockhoale/ukit@3.4.31.
See all versions of @ngockhoale/ukit for other recorded verdicts.
- Review the evidence and your use of @ngockhoale/ukit@3.4.31 before allowing it. Restrict the permissions described in this report.
- Choose an independently verified alternative or release. This report does not establish that other versions are safe.
Evidence last updated: .
Think this verdict is wrong? Dispute it on GitHub or read how reports are produced.