fix(agent): add audit pagination smoke #153

Merged
pdurlej merged 1 commit from codex/audit-pagination-smoke into main 2026-06-03 19:42:04 +02:00
Collaborator

Summary

  • adds simple Older/Newer cursor navigation to the Agent audit settings view
  • keeps the default page bounded at 50 rows and resets pagination when filters/workspace change
  • adds a router regression test proving cursorBefore is passed through and paginated audit rows stay redacted by default

Scope

Small UI/API test hardening for #119.

Non-goals

  • no production smoke run
  • no deploy, restart, migration, token rotation, or Infisical change
  • no audit raw payload broadening

Spec sources read

  • apps/web/src/views/settings/AgentAuditSettings.tsx
  • packages/api/src/routers/agent.ts
  • packages/db/src/repository/agent.repo.ts
  • packages/api/src/routers/agent.test.ts

Tests / smoke

  • pnpm --filter @kan/api test -- src/routers/agent.test.ts -t "paginates agent audit events"
  • pnpm --filter @kan/api test -- src/routers/agent.test.ts
  • pnpm --filter @kan/api typecheck
  • pnpm exec prettier --check apps/web/src/views/settings/AgentAuditSettings.tsx packages/api/src/routers/agent.test.ts
  • pnpm exec eslint src/views/settings/AgentAuditSettings.tsx from apps/web

Known existing repo checks:

  • pnpm --filter @kan/web typecheck still fails on existing bootstrap/workspace/cardPrefix/OperationalCockpitSettings errors outside this PR.
  • pnpm --filter @kan/web lint still fails on an existing ESLint 9 / @next/next/no-page-custom-font compatibility error in PageHead.tsx. The targeted changed-file eslint check passes.

Rollback

Revert this commit. Audit API remains usable without UI pagination.

Owner gates

None for merge.

Fixes #119

## Summary - adds simple Older/Newer cursor navigation to the Agent audit settings view - keeps the default page bounded at 50 rows and resets pagination when filters/workspace change - adds a router regression test proving `cursorBefore` is passed through and paginated audit rows stay redacted by default ## Scope Small UI/API test hardening for #119. ## Non-goals - no production smoke run - no deploy, restart, migration, token rotation, or Infisical change - no audit raw payload broadening ## Spec sources read - `apps/web/src/views/settings/AgentAuditSettings.tsx` - `packages/api/src/routers/agent.ts` - `packages/db/src/repository/agent.repo.ts` - `packages/api/src/routers/agent.test.ts` ## Tests / smoke - `pnpm --filter @kan/api test -- src/routers/agent.test.ts -t "paginates agent audit events"` - `pnpm --filter @kan/api test -- src/routers/agent.test.ts` - `pnpm --filter @kan/api typecheck` - `pnpm exec prettier --check apps/web/src/views/settings/AgentAuditSettings.tsx packages/api/src/routers/agent.test.ts` - `pnpm exec eslint src/views/settings/AgentAuditSettings.tsx` from `apps/web` Known existing repo checks: - `pnpm --filter @kan/web typecheck` still fails on existing bootstrap/workspace/cardPrefix/OperationalCockpitSettings errors outside this PR. - `pnpm --filter @kan/web lint` still fails on an existing ESLint 9 / `@next/next/no-page-custom-font` compatibility error in `PageHead.tsx`. The targeted changed-file eslint check passes. ## Rollback Revert this commit. Audit API remains usable without UI pagination. ## Owner gates None for merge. Fixes #119
fix(agent): add audit pagination smoke (#119)
All checks were successful
Forgejo CI / verify (pull_request) Successful in 42s
fbf5e85c68
pdurlej deleted branch codex/audit-pagination-smoke 2026-06-03 19:42:04 +02:00
Sign in to join this conversation.
No reviewers
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!153
No description provided.