Repository navigation
[scanner] resolver comments claim a stdin ('-') array source that is not implemented #305
Description
src/resolver.ts documents a stdin array source that does not exist:
- header (line 3):
Handles array expansion (comma-separated, globs, @file, stdin). - line 55–56:
// - (stdin) handled at CLI level before thisfollowed byif (raw === '-') return [raw];
Nothing in src/cli.ts reads stdin for a - value; resolver.test.ts:189-190 pins --pa_f=- as the literal string -, and README/HELP list only comma, glob and @file sources. So a user who reads the module comment and tries --file=- with piped input gets a run against the literal -.
Two consistent options; either is fine, pick the smaller unless stdin is wanted:
- A. drop
stdinfrom the header and replace the line 55 comment with "a bare-is a literal value, not stdin" (docs/comment only). - B. implement
-→ one value per non-empty stdin line (same shape as@file), and document it in HELP/README.
Decision taken by hive: proposing A (comment-only) unless a maintainer wants B.
— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88
Activity
hivecommons-hive commentedon Oct 21, 2026
🔒 Claimed by @mendezr (contributor on hive hosted-available-oke-11-placeholder-r05x). The claim lasts 30m0s without activity and renews while the work continues. Reply
/unclaimto release, or/claimto take it over if you outrank the holder.docs(resolver): sync declaration header
hivecommons-hive commentedon Oct 21, 2026
Folded finding. This was filed as a separate issue, but 1 open agent-filed issue (plus 6 closed as completed in the last 14 days) already cite
src/resolver.ts, so it was folded into the oldest of them instead of opening another. Treat this issue as the tracker for that component: prefer one structural fix that covers every folded variant over a PR per variant.Review backlog (intent-alignment): Test title still calls a bare '-' a 'stdin placeholder'
Filed from an out-of-scope finding in the Hive review of PR #306.
PR: #306 (#306)
Perspective: intent-alignment
Severity: info (P3)
Evidence:
src/resolver.test.ts:188(promptargs/src/resolver.test.ts
Line 188 in 70db3d2
Finding: Test title still calls a bare '-' a 'stdin placeholder'
src/resolver.test.ts:188 is titled 'keeps "-" (stdin placeholder) as a single literal'. After this PR the module states a bare '-' is literal and not stdin (src/resolver.ts:55), so the 'stdin placeholder' wording is the last residual hint of the unimplemented source. Issue #305 only cited resolver.ts lines 3 and 55-56, so this is adjacent cleanup, not a gap in the fix.
Fingerprint:
b8f66cd9ac462dfa0b06464c58733e93a4049430ef439e446f18e3a5f0dfa51efiled by Hive review backlog
hivecommons-hive commentedon Oct 21, 2026
Linked again from the Hive review of PR #306.
docs(resolver): clarify that a bare dash is literal (#306)