@ngockhoale/ukit@3.4.26
Install/update an index-first AI workspace for Claude Code, OpenAI Codex and omp (Oh My Pi).
AI Security Review
scanned 15d ago · by lpm-firewall-aiThe credential forwarding path creates a concrete conditional exposure because a configured endpoint is not destination-restricted. The feature is stage-gated and user-configurable, so this supports a warning rather than a finding of malicious credential theft.
Static reason
No blocking static signals were detected.
Trigger
A runtime caller enables the decision stage and submits a non-empty batch while a shared gateway credential is available.
Impact
A misconfigured or untrusted endpoint could receive the shared credential. This is conditional exposure, not evidence of credential theft.
Mechanism
The client resolves the endpoint and credential separately, then sends the credential in the request headers; the source shows no destination allowlist.
AI rationale
The credential forwarding path creates a concrete conditional exposure because a configured endpoint is not destination-restricted. The feature is stage-gated and user-configurable, so this supports a warning rather than a finding of malicious credential theft.
Network endpoints2
openai.unicjsc.comUKIT_DECISION_BASE_URL-configured endpoint
Decision evidence
public snapshotAI called this Suspicious at 84.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- The decision endpoint can come from an environment variable or user configuration without a destination allowlist.
- When no decision key is set, the client can fall back to the existing shared gateway token.
- The decision client combines that credential with the independently configured endpoint and sends it in an authenticated request.
- A sensitive-value gate screens serialized request state, but does not bind the authentication header to an approved destination.
- The request path is stage-gated and the decision client can be disabled by configuration.
- No confirmed malicious intent or credential theft was established; the exposure depends on endpoint configuration.
Evidence against
- The default decision endpoint is the package’s named provider, and the runtime path is opt-in through configuration.
- The package exposes a CLI and has no npm install lifecycle hook in its manifest.
Behavioral surface
ChildProcessCryptoDynamicRequireEnvironmentVarsFilesystem
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
MediumAi Review Evidencesrc/core/gatewayProbe.js
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.26.
See all versions of @ngockhoale/ukit for other recorded verdicts.
- Review the evidence and your use of @ngockhoale/ukit@3.4.26 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.