Skip to content

Ines - #2

Merged
inesaranab merged 2 commits into
mainfrom
ines
Jul 24, 2026
Merged

Ines#2
inesaranab merged 2 commits into
mainfrom
ines

Conversation

@inesaranab

@inesaranab inesaranab commented Jul 24, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Documentation

    • Added comprehensive guidance for FastAPI, Pydantic, Instructor, and screening conventions.
    • Documented routing, dependency injection, response handling, streaming, validation, security, testing, and recommended tools.
    • Added attribution and licensing context for included skills.
  • Configuration

    • Enabled FastAPI tooling to discover the application entry point.
    • Updated ignore rules so authored skills can be tracked.
  • Refactor

    • Modernized API dependency and header declarations without changing endpoint behavior.

Annotated[T, Depends(...)] moves the dependency out of the argument
default, so ruff's B008 no longer fires and the extend-immutable-calls
exemption is no longer needed.
Adds .claude/skills/ — the SKILL.md packets that steered the AI-assisted
development, and un-ignores that folder (local Claude settings stay ignored).

- screening-conventions: original — the project's hexagonal boundary, trust
  model (JD trusted / transcript untrusted), fail-closed guardrail behaviour,
  and test-first workflow.
- instructor: adapted from upstream docs/CLAUDE.md.
- fastapi, pydantic: maintainer-authored skills, vendored verbatim under their
  MIT licenses (attribution in .claude/skills/README.md).
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5f6495d1-5579-40d9-8536-44bcaa41cd27

📥 Commits

Reviewing files that changed from the base of the PR and between 1a67ef7 and f01b1c5.

📒 Files selected for processing (14)
  • .claude/skills/README.md
  • .claude/skills/fastapi/SKILL.md
  • .claude/skills/fastapi/references/dependencies.md
  • .claude/skills/fastapi/references/other-tools.md
  • .claude/skills/fastapi/references/path-operations.md
  • .claude/skills/fastapi/references/pydantic.md
  • .claude/skills/fastapi/references/responses.md
  • .claude/skills/fastapi/references/streaming.md
  • .claude/skills/instructor/SKILL.md
  • .claude/skills/pydantic/SKILL.md
  • .claude/skills/screening-conventions/SKILL.md
  • .gitignore
  • app/api/main.py
  • pyproject.toml

📝 Walkthrough

Walkthrough

Added agent-skill documentation for screening, FastAPI, Instructor, and Pydantic; updated .claude tracking rules; migrated API dependency declarations to Annotated; and configured the FastAPI application entrypoint.

Changes

Agent skills documentation

Layer / File(s) Summary
Skills catalog and screening conventions
.claude/skills/README.md, .claude/skills/screening-conventions/SKILL.md, .gitignore
Documents available skills, attribution, screening-service conventions, and tracking rules for .claude/skills/.
FastAPI endpoint and dependency guidance
.claude/skills/fastapi/SKILL.md, .claude/skills/fastapi/references/dependencies.md, .claude/skills/fastapi/references/path-operations.md, .claude/skills/fastapi/references/responses.md
Adds guidance for FastAPI dependency injection, routing, response typing, and one-operation-per-function handlers.
FastAPI data, streaming, and tooling references
.claude/skills/fastapi/references/pydantic.md, .claude/skills/fastapi/references/streaming.md, .claude/skills/fastapi/references/other-tools.md
Documents Pydantic annotation patterns, streaming and SSE responses, binary streaming, and recommended tools.
Instructor and Pydantic skills
.claude/skills/instructor/SKILL.md, .claude/skills/pydantic/SKILL.md
Adds guidance for validated LLM outputs and Pydantic modeling, validation, coercion, recursive types, and polymorphic models.

FastAPI application wiring

Layer / File(s) Summary
Annotated dependency wiring and entrypoint configuration
app/api/main.py, pyproject.toml
Migrates API key and service dependencies to Annotated, preserves endpoint behavior, and sets app.api.main:app as the FastAPI entrypoint.

Estimated code review effort: 2 (Simple) | ~15 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ines

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@inesaranab
inesaranab merged commit 6c86ec8 into main Jul 24, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant