Skip to content

knowledge: swap real addresses for documentation values in pasted CLI output (npm always packs README) - #331

Open
choiyounggi wants to merge 1 commit into
mainfrom
knowledge/autoflush-20261104-133008-18720
Open

choiyounggi wants to merge 1 commit into
mainfrom
knowledge/autoflush-20261104-133008-18720

Conversation

@choiyounggi

Copy link
Copy Markdown
Owner

Knowledge flush — 1 insight(s)

Headless autoflush run, 2026-11-04 (claims by run 20261104-133008-18720). Claimed 4 candidates: ingested 1, released 3 (2 pending duplicates of open PRs, 1 unreadable in this run).

Verified best-practice

63c0a85e1e6199a2 — real environment values in a README's pasted CLI output (ingested)

  • Claim: before committing a document that contains a real command run, replace addresses, hostnames and account names with documentation values, then git grep -F each real value to zero hits. Reason: npm always packs README, and a published version cannot be changed.
  • Sources checked (via WebSearch; this run has no WebFetch):
  • Changed from the candidate: the candidate suggested the Tailscale-looking example 100.64.0.10. That is an address a real tailnet device can hold (Tailscale IP pool docs), so the page recommends RFC 5737 documentation addresses instead and lists the CGNAT shape as an edge row.
  • Confidence: verified (the npm packing and immutability rules and RFC 5737 come from official docs; the grep step follows from them and was carried out in the field).

b5863e2d758c98fb — Claude Code behind a gateway still sends bootstrap requests while a claude.ai OAuth login remains (released, not verified in this run)

126503dbe78559da — claude-code-action: --allowedTools does not remove Agent; deny it with --disallowedTools (released, not verified in this run)

fc76dc4dfa08ecde (released): this headless run cannot list ~/.dev-loop/queue. Queue files can only be opened by guessing session ids from ~/.claude/history.jsonl, and this row's file was not found that way. Left for the manual skill.

Existing-layer check

Pages read: qa-deliverables-command-transcripts-in-a-document, qa-process-session-identity-leak-in-plugin-prose, security-data-commit-identity-in-public-repos

  • wiki_search ("real IP addresses or account names in README example output published to npm…" and "pasting real terminal output into documentation…") returned qa-deliverables-command-transcripts-in-a-document, security-data-commit-identity-in-public-repos, security-secrets-secrets-in-code and security-data-masking-verification.
  • qa-deliverables-command-transcripts-in-a-document already owns "pasting a captured CLI run into a README". Its edge table replaced per-run values (timings, temp paths) with placeholders, but it said nothing about environment values or published copies. Merged there: Do-this step 6, 3 edge rows, 1 Instead-of row, 5 source bullets, and a wider "When this applies". Body is 83 lines.
  • security-data-commit-identity-in-public-repos covers an author address already pushed ("treat it as disclosed"). The new edge row for an already-published value links to it and does not repeat it. No conflict.
  • qa-process-session-identity-leak-in-plugin-prose covers session-injected names in plugin prose. Its edge row already extends to hostnames. This is a sibling check with a different trigger (a review diff vs a pasted transcript), so it is linked, not merged.
  • security-secrets-secrets-in-code (from search snippets only) is about credentials and leak response. An address or account name is not a credential, so it is not the right home. Not modified.
  • Also read wiki/qa/index.md and wiki/security/index.md for routing; the qa index load-when line was extended.
  • related += security-data-commit-identity-in-public-repos, qa-process-session-identity-leak-in-plugin-prose. last_verified set to 2026-11-04. log.md revise entry appended.

Open-PR check

20 open knowledge/* PRs: #330, #329, #328, #324, #319, #318, #317, #315, #310, #309, #308, #307, #306, #305, #304, #301, #300, #299, #298, #293. Changed-file lists and PR bodies came from gh pr list --json files,body. This headless run cannot fetch or diff branch content.

Routing decision

  • 63c0a85e1e6199a2 → qa/deliverables/command-transcripts-in-a-document.md (merge). The candidate's domain hint was security, but the artifact being produced is a document containing a pasted transcript, and that page owns it. The security pages own credentials (secrets-in-code) and commit identity, not example values in docs. The directive is general (any published README or doc), not specific to one project.
  • No new category.

Local-layer candidates

none

Queue bookkeeping

  • Retired in place (status: retired, retiredBy, outcome), the same way earlier headless runs retired rows in that session file: 63c0a85e1e6199a2.
  • Released back to pending: b5863e2d758c98fb, 126503dbe78559da, fc76dc4dfa08ecde. Confirmed with queue-claim.js list, which printed exactly those three ids.

@choiyounggi choiyounggi added the dev-loop:knowledge Auto-opened wiki knowledge ingest PR (owner reviews & merges) label Nov 4, 2026
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Wiki agent gate: error — the agent run produced no verdict (failed closed). See the workflow log.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev-loop:knowledge Auto-opened wiki knowledge ingest PR (owner reviews & merges)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant