chore: migrate @app.on_event('startup') to FastAPI lifespan handler #70

New issue

Closed

opened 2026-04-05 20:59:39 -07:00 by pyr0ball · 1 comment

pyr0ball commented

2026-04-05 20:59:39 -07:00

Owner

Summary

dev_api.py uses the deprecated @app.on_event('startup') pattern (line ~142), which triggers a deprecation warning in recent FastAPI versions.

Change

Migrate to the lifespan context manager pattern:

from contextlib import asynccontextmanager

@asynccontextmanager
async def lifespan(app: FastAPI):
    # startup logic here
    yield
    # shutdown logic here (if any)

app = FastAPI(title="Peregrine Dev API", lifespan=lifespan)

Priority

Low — deprecation warning only, no functional impact yet.

## Summary `dev_api.py` uses the deprecated `@app.on_event('startup')` pattern (line ~142), which triggers a deprecation warning in recent FastAPI versions. ## Change Migrate to the `lifespan` context manager pattern: ```python from contextlib import asynccontextmanager @asynccontextmanager async def lifespan(app: FastAPI): # startup logic here yield # shutdown logic here (if any) app = FastAPI(title="Peregrine Dev API", lifespan=lifespan) ``` ## Priority Low — deprecation warning only, no functional impact yet.

pyr0ball commented

2026-04-05 23:51:00 -07:00

Author

Owner

Migrated in c0649c7. @app.on_event("startup") replaced with @asynccontextmanager async def lifespan(app). Startup logic unchanged; no shutdown handler needed. All 671 tests pass.

Migrated in c0649c7. `@app.on_event("startup")` replaced with `@asynccontextmanager async def lifespan(app)`. Startup logic unchanged; no shutdown handler needed. All 671 tests pass.