gemini(w4): analytics API no-internal-id response audit #124

Closed
opened 2026-05-28 01:20:42 +02:00 by codex · 1 comment
Collaborator

Parent: #2
Agent lane: Gemini 3.5 Flash
Wave: 4 / analytics foundation
Risk class: low

Goal

Catch internal ID leaks before analytics becomes relied upon.

Context refs

  • AGENTS.md security/public ID rule
  • docs/agent-mcp-contract.md

Scope

  • Inspect analytics, transition history, recent activity, and flow health endpoints.
  • Search for returned raw id, cardId, listId, workspaceId.
  • Open/patch only exact leaks.

Acceptance

  • Audit list includes each endpoint checked.
  • No raw internal IDs in external API/MCP response contracts.
  • False positives are noted.

Suggested checks

  • Static search plus targeted response tests if practical.

Non-goals / fences

  • Do not deploy, restart production, rotate secrets, or run production migrations.
  • Do not widen MCP write authority or public exposure.
  • Keep the change small enough for one focused PR or one scouting report.

Expected output

A short PR or issue comment with findings, touched files, tests run, and remaining risks.

Parent: #2 Agent lane: Gemini 3.5 Flash Wave: 4 / analytics foundation Risk class: low ## Goal Catch internal ID leaks before analytics becomes relied upon. ## Context refs - `AGENTS.md` security/public ID rule - `docs/agent-mcp-contract.md` ## Scope - Inspect analytics, transition history, recent activity, and flow health endpoints. - Search for returned raw `id`, `cardId`, `listId`, `workspaceId`. - Open/patch only exact leaks. ## Acceptance - Audit list includes each endpoint checked. - No raw internal IDs in external API/MCP response contracts. - False positives are noted. ## Suggested checks - Static search plus targeted response tests if practical. ## Non-goals / fences - Do not deploy, restart production, rotate secrets, or run production migrations. - Do not widen MCP write authority or public exposure. - Keep the change small enough for one focused PR or one scouting report. ## Expected output A short PR or issue comment with findings, touched files, tests run, and remaining risks.
Author
Collaborator

Codex PR opened: #149

Scope:

  • added docs/audits/analytics-public-id-response-audit.md,
  • checked getRecentActivity, getTransitionHistory, getFlowMetrics, getBoardFlowMetrics, and MCP wrappers,
  • found no raw internal id, cardId, listId, boardId, or workspaceId in the audited response contracts,
  • documented follow-up guardrail for future response-level tests.

Checks:

  • static search over agent router, MCP wrappers, and integration tests,
  • pnpm exec prettier --check docs/audits/analytics-public-id-response-audit.md — passed.

No production deploy, restart, DB migration, token/Infisical change, or runtime/API behavior change.

Codex PR opened: https://git.pdurlej.com/pdurlej/kan-ductor/pulls/149 Scope: - added `docs/audits/analytics-public-id-response-audit.md`, - checked `getRecentActivity`, `getTransitionHistory`, `getFlowMetrics`, `getBoardFlowMetrics`, and MCP wrappers, - found no raw internal `id`, `cardId`, `listId`, `boardId`, or `workspaceId` in the audited response contracts, - documented follow-up guardrail for future response-level tests. Checks: - static search over agent router, MCP wrappers, and integration tests, - `pnpm exec prettier --check docs/audits/analytics-public-id-response-audit.md` — passed. No production deploy, restart, DB migration, token/Infisical change, or runtime/API behavior change.
Sign in to join this conversation.
No labels
3plus3-followup
agent/claude-code
agent/codex
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
analytics
api
cockpit
dependency/blocked
dependency/blocks-others
dependency/cross-repo
dependency/needs-confirmation
docs
domain:agents
domain:ci
domain:docs
domain:forgejo
domain:infra
domain:memory
domain:runtime
domain:signal
domain:ux
flow/architecture
flow/blocked
flow/deployed
flow/done
flow/implementation
flow/intake
flow/maintained
flow/observed
flow/ready
flow/refining
flow/retired
flow/review
gemini-flash
judge/codex-candidate
judge/hermes-candidate
judge/low-confidence
judge/needs-refinement
judge/operator-needed
judge/p0
judge/p1
judge/p2
judge/p3
judge/park
judge/patchwarden-candidate
judge/stale-priority
kind/adr
kind/bug
kind/chore
kind/feature
kind/infra
kind/ops
kind/refactor
kind/research
leviathan
mcp
merge/auto
merge/manual
merge/manual-dependency-conflict
merge/manual-failing-tests
merge/manual-merge-conflict
merge/manual-missing-review
merge/manual-operator-preference
merge/manual-red-zone
merge/manual-security-sensitive
merge/manual-unclear-scope
merge/manual-unknown
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
ops
priority:p0
priority:p1
priority:p2
priority:p3
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
safety
safety:external-write
safety:no-prod-mutation
safety:prod-impact
safety:secret-touch
scout
security
size/large
size/medium
size/small
size/tiny
size/unknown
small-task
source/adr
source/agent-generated
source/manual
source/operator-chat
source/voice-note
status:blocked
status:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:operator-needed
status:parked
tests
tier:0-platform-substrate
tier:1-iskra-value-layer
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
ui
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pdurlej/kan-ductor#124
No description provided.