Skip to content
Back to skills

Code Review Langfuse Langfuse

ASecurity

Review Langfuse code changes for correctness, regressions, and best practices.

  • 2 stars
  • 0 votes
  • 0 copies
  • 1 view
  • Added October 17, 2026
code-qualitycode-reviewapibackendsecurityperformance

Works with

  • cli
  • api

Security analysis

A100/100

Scanned October 17, 2026

npx -y skills add karanmrn/karanagentskills --skill code-review--langfuse--langfuse --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of Code Review Langfuse Langfuse?

Add the live security badge to your README. It updates with every re-scan.

Security grade badge for Code Review  Langfuse  Langfuse
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/karanmrn-code-review-langfuse-langfuse/badge)](https://www.skillsdirectory.com/skills/karanmrn-code-review-langfuse-langfuse)

More formats (shields.io, HTML) on the badges page. Keep it an A: scan every change in CI with Pro.

SKILL.md
---
name: code-review
description: Review Langfuse code changes for correctness, regressions, and best practices.
---

# Code Review

Use this skill when the task is to review code changes rather than implement a
feature.

## Start Here

- Read [`references/review-checklist.md`](references/review-checklist.md) for
  the repo's canonical review rules.
- Read root [`AGENTS.md`](../../../AGENTS.md) and the nearest package
  `AGENTS.md` for the files under review.
- If the review touches ClickHouse, also use the shared
  `clickhouse-best-practices` skill.
- If the review touches backend code, also use the shared
  `backend-dev-guidelines` skill where relevant.
- If the change accepts a user-supplied URL, adds outbound HTTP, introduces a
  new integration, or touches secrets, RBAC, or redirect handling, also use
  the shared [`security-review`](../security-review/SKILL.md) skill. Run its
  [`references/checklist.md`](../security-review/references/checklist.md)
  before signoff.

## Review Priorities

Focus on:

- correctness bugs
- behavioral regressions
- security and tenant-isolation risks
- performance issues with real impact
- missing or weak tests for risky changes
- Before calling coverage missing, identify the unique regression each proposed
  test catches. Prefer merging into the closest existing suite. Flag repeated
  tests of the same predicate across layers unless each proves a distinct
  transport, projection, or execution boundary.

## Output Expectations

- Findings first, ordered by severity
- File and line references for each finding
- Short summary only after findings
- If no findings, say so explicitly and mention any residual risk or coverage gaps

## Scope Guidance

Use `references/review-checklist.md` for Langfuse-specific checks such as:

- ClickHouse and Postgres migration expectations
- project-scoped tenant isolation checks
- API/Fern consistency
- banner-offset UI positioning
- environment variable access patterns

Do not duplicate those rules in ad hoc prompts or tool-specific command files.

Attribution

Is this your skill, or is something wrong with this listing? Request removal or report an issue. Author removals are honored within 72 hours.

Comments

Loading comments…

Newsletter

Get the weekly list of safe new skills, and the ones we pulled.