Skip to content

ci(profiles): W4 add coverage for 9 orphan extensions + mlops - #187

Merged
ulises-jeremias merged 1 commit into
mainfrom
ci/w4-profiles-coverage
Aug 23, 2026
Merged

ci(profiles): W4 add coverage for 9 orphan extensions + mlops#187
ulises-jeremias merged 1 commit into
mainfrom
ci/w4-profiles-coverage

Conversation

@ulises-jeremias

Copy link
Copy Markdown
Member

Description

Fixes orphan extension coverage (W4-1, §3). 9/18 extensions had no L3 profile (cors, redis, sqlalchemy, otel, rate-limit, mlflow-tracing, rag-pgvector, django-spectacular, pre-commit). Adds 4 curated profiles covering 100% of extensions plus mlops-sklearn template (PR #179).

Changes

  • ci/profiles/fastapi-observability.jsonfastapi-starter + github-setup + fastapi-cors (security) + fastapi-sqlalchemy (database) + fastapi-opentelemetry (observability) + fastapi-rag-pgvector (ai-ml) — 5 distinct categories, all fastapi-backend compatible
  • ci/profiles/fastapi-data.jsonfastapi-starter + pre-commit (ci) + fastapi-redis (database) + fastapi-rate-limit (security) + fastapi-mlflow-tracing (observability) — respects one-per-category, covers remaining fastapi orphans and pre-commit (cannot co-locate github-setup + pre-commit — same ci category)
  • ci/profiles/django-full.jsondjango-api + github-setup + development-container + django-docker + postgres + django-spectacular — covers django-spectacular orphan (observability, django-backend only)
  • ci/profiles/mlops-sklearn-default.jsonmlops-sklearn-starter + github-setup — cherry-picked from PR ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179 (8e581e6); will be deduplicated when ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179 merges. Note: task suggested all-github-setup + all-pre-commit but they share ci category — invalid per assert_profile_valid (category uniqueness). pre-commit orphan is already covered via fastapi-data.
  • templates.json — add mlops-sklearn to github-setup type array for L2/L3 coverage (as per ci: add mlops-sklearn-starter profile and github-setup support (fixes #92) #179)

Category / Type Validation

Each profile validated via assert_profile_valid:

How Has This Been Tested?

  • python scripts/ci/validate-registry.py — ✅ registry ok (6 templates, 18 extensions)
  • python scripts/ci/generate-matrix.py --layer validate-profiles — ✅ 13 profiles validated (9 → 13)
  • Orphan check: 0 orphans (was 9) — covered = 18/18
  • python scripts/ci/generate-matrix.py --layer profiles — generates valid L3 matrix

Type of Change

  • CI / tooling

Checklist

  • url points to correct path on main
  • slug globally unique
  • All required fields present
  • Extension type is array where multi-type

- Add fastapi-observability (github-setup + fastapi-cors + fastapi-sqlalchemy + fastapi-opentelemetry + fastapi-rag-pgvector) covering 4 orphans with distinct categories (ci/security/database/observability/ai-ml)
- Add fastapi-data (pre-commit + fastapi-redis + fastapi-rate-limit + fastapi-mlflow-tracing) covering remaining 4 fastapi orphans (ci/database/security/observability) — respects category uniqueness (one per category)
- Add django-full (github-setup + development-container + django-docker + postgres + django-spectacular) covering django-spectacular orphan
- Add mlops-sklearn-default (mlops-sklearn-starter + github-setup) as per PR #179 — cherry-picked profile and github-setup type extension for mlops-sklearn; pre-commit orphan is covered via fastapi-data (cannot co-locate ci category with github-setup in same profile)
- Update templates.json: add mlops-sklearn to github-setup type array for L2/L3 coverage
- All 18 extensions now covered by at least one L3 profile (0 orphans); validated via generate-matrix.py --layer validate-profiles and validate-registry.py

Fixes orphan coverage noted in W4-1 and §3 (9/18 extensions orphan: cors, redis, sqlalchemy, otel, rate-limit, mlflow-tracing, rag-pgvector, django-spectacular, pre-commit)
@ulises-jeremias
ulises-jeremias marked this pull request as ready for review August 23, 2026 18:18
@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@ulises-jeremias, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 42 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b675a552-b713-42fd-a53e-ec4c7e6fc208

📥 Commits

Reviewing files that changed from the base of the PR and between 1e8f5d0 and 6115362.

📒 Files selected for processing (5)
  • ci/profiles/django-full.json
  • ci/profiles/fastapi-data.json
  • ci/profiles/fastapi-observability.json
  • ci/profiles/mlops-sklearn-default.json
  • templates.json

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.

@ulises-jeremias
ulises-jeremias merged commit 3dbe7d8 into main Aug 23, 2026
57 checks passed
@ulises-jeremias
ulises-jeremias deleted the ci/w4-profiles-coverage branch August 23, 2026 18:33
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