Skip to content

feat(forge): migrate FastAPI request validation#14

Merged
fronzec merged 1 commit into
feat/fastapi-project-migrationfrom
feat/fastapi-request-validation
Jul 19, 2026
Merged

feat(forge): migrate FastAPI request validation#14
fronzec merged 1 commit into
feat/fastapi-project-migrationfrom
feat/fastapi-request-validation

Conversation

@fronzec

@fronzec fronzec commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Closes #12

Summary

  • Migrate the synchronized FastAPI request-validation tutorial into Forge's concept-first catalog.
  • Preserve typed path, query, and body examples with focused reproducible requests.
  • Fix the confirmed optional-body None dereference and pin the historical dependency set.

Changes

Area Change
web-framework-basics/request-validation/python-fastapi/ Add the validation handlers, requests, dependencies, and experiment record.
README.md Add the experiment to the Forge catalog.

Test Plan

  • Installed the pinned dependencies in an external Python 3.11 environment.
  • Ran uv pip check; all installed packages are compatible.
  • Compiled and imported main.py.
  • Verified GET /hello/Forge and the bodyless optional-item update.
  • Verified invalid constrained parameters return HTTP 422.

Chain Context

Field Value
Chain FastAPI source migration
Tracker PR Not needed
Position 2 of 2
Base feat/fastapi-project-migration
Depends on #13
Follow-up None
Review budget 329 / 400 changed lines
Starts at The DNF history migration branch plus source commit 7a35799
Ends with A standalone request-validation experiment in Forge

Chain Overview

main
 └── #13 FastAPI DNF history
      └── 📍 #14 FastAPI request validation

Scope

  • Includes: the tutorial source snapshot, reproducibility cleanup, and the optional-body fix.
  • Excludes: the DNF history experiment, source archival, and Forge merge.

Autonomy

  • CI is expected to pass for this PR branch.
  • This PR has one deliverable scope.
  • This PR can be rolled back by removing its experiment and catalog row.
  • Runtime verification and documentation cover this unit.

Contributor Checklist

fronzec commented Jul 19, 2026

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fronzec fronzec added the type:feature New feature label Jul 19, 2026
@fronzec
fronzec merged commit b3980bc into feat/fastapi-project-migration Jul 19, 2026
2 checks passed
fronzec added a commit that referenced this pull request Jul 19, 2026
Closes #12

## Summary

- Publish the already-reviewed FastAPI request-validation snapshot to `main`.
- Recover the child slice after PR #14 was merged into the already-merged PR #13 branch.
- Preserve the exact six-file, 329-line payload from commit `9e691e9`.

## Why This PR Exists

PR #13 merged into `main` first. PR #14 still targeted `feat/fastapi-project-migration`, so merging #14 updated that branch but did not place its snapshot on trunk. GitHub reported #14 as merged while `main...9e691e9` remained diverged and issue #12 stayed open.

## Changes

| Area | Change |
| --- | --- |
| `web-framework-basics/request-validation/python-fastapi/` | Reapply the exact validated request-validation snapshot. |
| `README.md` | Restore its Forge catalog entry. |

## Test Plan

- [x] Confirmed PR #14's commit is not contained in `main`.
- [x] Reapplied only commit `9e691e9` onto current `main`.
- [x] Verified `git diff --check` and the expected 6-file, 329-line scope.
- [x] Reused the dependency, import, and live HTTP evidence recorded in the snapshot README and PR #14.

## Contributor Checklist

- [x] Linked approved issue #12.
- [x] Used conventional commit format.
- [x] Kept the recovery payload identical to the reviewed child slice.
- [x] Added no AI attribution or co-author trailers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant