@ngockhoale/ukit@3.4.30
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-aiThe inspected source does not establish malicious behavior or an automatic install attack. The decision client has a concrete conditional credential-forwarding risk because a configured endpoint is not visibly bound to the shared gateway token’s intended service.
Static reason
No blocking static signals were detected.
Trigger
A runtime decision request when the client has a configured endpoint and gateway token.
Impact
A misconfigured or untrusted endpoint could receive the gateway token.
Mechanism
The client resolves the endpoint and API key separately, adds the key to request headers, and sends a POST.
AI rationale
The inspected source does not establish malicious behavior or an automatic install attack. The decision client has a concrete conditional credential-forwarding risk because a configured endpoint is not visibly bound to the shared gateway token’s intended service.
Evidence
src/core/gatewayProbe.jssrc/decision/client.js
Network endpoints1
openai.unicjsc.com
Decision evidence
public snapshotAI called this Suspicious at 83.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- The manifest exposes a CLI and lists no npm lifecycle hook; the package has no automatic install trigger.
- The watchdog and decision-coverage files describe a task hook and a CI coverage guard; the probe, benchmark and performance targets are tooling.
- The install helpers handle project files, protect agent paths, preserve settings and snapshot managed files.
- The decision client obtains an endpoint and API key through separate resolvers, then sends a POST request.
- The endpoint resolver accepts a configured URL, and the key resolver can use the existing shared gateway token; no destination allowlist is evident in the inspected code.
Evidence against
- A configured decision endpoint can receive the shared gateway token when the decision client sends its request. The inspected resolver does not restrict that endpoint to the token’s intended service.
Behavioral surface
ChildProcessCryptoDynamicRequireEnvironmentVarsFilesystemShell
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.30.
See all versions of @ngockhoale/ukit for other recorded verdicts.
- Review the evidence and your use of @ngockhoale/ukit@3.4.30 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.