d-oit/do-epub-studio·#1026· closed

fix(security): enforce schema validation for admin insights query params

Analysed against 9177c7e93 files changedby d-oit
Open PR on GitHub

Change-risk score

4.8out of 10Typical
Repository health
9.3to9.4base against this head

This change sits in the middle third of this repository's own risk distribution, so it is about as risky as the work around it. It is riskier than 44% of this repository's own commits.

No changed file lost health in this diff. Nothing outside this PR depends on a contract it changed.

Files changed
3
in this diff
Contracts changed
0
removed or signature
Outside callers
0
not in this diff
Findings added
0
2 resolved

The repository map needs the file index from this snapshot, which is not available yet. Everything below is unaffected.

What this PR added, and what it took away

Introduced findings are the ones inside the lines this PR added, not the file's existing ones. The resolved column is the only place in the product that lists them.

Introduced (0)
  • Nothing added in the changed lines.
Resolved (2)
  • dry violation
    apps/worker/src/routes/admin/insights.ts
  • dry violation
    packages/schema/src/schemas/insights.ts

Who else knows this code

The primary author of each changed file by share of its recent commits, mined from git history. Advisory: it says who has context, not who must review.

  • apps/worker/src/__tests__/routes.admin.insights.test.tsDominik Oswald 100% of 1 commits

AI-authored against human-authored

Each changed file attributed to whichever author class contributed more of its added lines, then scored. File-level, because a line-level blame index is not persisted.

Files changed
AI 3Human 0
Findings introduced
AI 0Human 0
Share of the regression
AI 0.00Human 0.00

Get this on your own pull requests

This page came from an index of d-oit/do-epub-studio. No model calls, no configuration, and it refreshes on every push. Install the bot and every pull request gets one of these.