Repository navigation
knowledge: swap real addresses for documentation values in pasted CLI output (npm always packs README) - #331
Open
choiyounggi wants to merge 1 commit into
Open
choiyounggi wants to merge 1 commit into
choiyounggi wants to merge 1 commit into
Conversation
Contributor
Wiki agent gate: error — the agent run produced no verdict (failed closed). See the workflow log. |
This was referenced Nov 4, 2026
knowledge: --allowedTools does not remove Agent; limit a CI agent runner with --disallowedTools
#332
Open
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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)
git grep -Feach real value to zero hits. Reason: npm always packs README, and a published version cannot be changed.files: "Certain files are always included, regardless of settings" (package.json, README, LICENSE/LICENCE,main,bin).git log -Gand a per-taggit grepin tags v0.2.0–v0.4.1 and in the npm 0.4.0/0.4.1 tarballs. The current tree had 0 hits.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.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:
--allowedToolsdoes not removeAgent; 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…") returnedqa-deliverables-command-transcripts-in-a-document,security-data-commit-identity-in-public-repos,security-secrets-secrets-in-codeandsecurity-data-masking-verification.qa-deliverables-command-transcripts-in-a-documentalready 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-reposcovers 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-prosecovers 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.wiki/qa/index.mdandwiki/security/index.mdfor routing; the qa index load-when line was extended.security-data-commit-identity-in-public-repos,qa-process-session-identity-leak-in-plugin-prose.last_verifiedset to 2026-11-04.log.mdrevise 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 fromgh pr list --json files,body. This headless run cannot fetch or diff branch content.wiki/qa/deliverables/orwiki/security/data/. knowledge: Gradle include filters, HMAC key length, Slack webhook probe, approved-command secrets, .env targets, worktree pre-push #300 editssecurity/secrets/secrets-in-code.mdwith Slack-webhook and approved-command-secret rows (per its body), which is a different trigger and a page this candidate does not touch. knowledge: fix Korean particles after rewriting a token in prose #308/knowledge: decide superseded sections from the amendments, skipping quoted OLD text #305 editqa/document-verification/pages on Korean particles and superseding records, which is unrelated.platforms/processes/spawning-an-agent-cli-from-a-server.mdwithANTHROPIC_BASE_URL/ANTHROPIC_AUTH_TOKENgateway rows, and knowledge: gateway alias resolution, Spring env-var names, CDN traffic path, stderr in captures #298 addsbackend/common/llm/gateway-model-alias-resolution.md. Same domain and trigger (Claude Code behind a gateway). This matches the verdict in knowledge: size a gate's per-CHECK timeout when one CHECK runs the suite N times #330's run, and both PRs are still open.platforms/tools/deny-rules-under-bypassed-permissions.md(Claude Code allow/deny rule semantics), and knowledge: child claude hooks/env (Stop per turn, --settings env, gateway proxy), Explore ignores CLAUDE_CODE_SUBAGENT_MODEL #328 editsplatforms/tools/per-call-subagent-effort.md. Both plausibly overlap--allowedTools/--disallowedToolsandAgent. Same verdict as knowledge: size a gate's per-CHECK timeout when one CHECK runs the suite N times #330's run.Routing decision
qa/deliverables/command-transcripts-in-a-document.md(merge). The candidate's domain hint wassecurity, 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.Local-layer candidates
none
Queue bookkeeping
status: retired,retiredBy,outcome), the same way earlier headless runs retired rows in that session file: 63c0a85e1e6199a2.queue-claim.js list, which printed exactly those three ids.