Skip to content

Narrow runtime placeholder lookup by route#1126

Merged
cbusillo merged 1 commit into
mainfrom
fix-runtime-placeholder-lookup-narrowing
Jun 3, 2026
Merged

Narrow runtime placeholder lookup by route#1126
cbusillo merged 1 commit into
mainfrom
fix-runtime-placeholder-lookup-narrowing

Conversation

@cbusillo
Copy link
Copy Markdown
Owner

@cbusillo cbusillo commented Jun 3, 2026

Summary

  • restore database-side route predicates for runtime placeholder retirement when context/instance are nonblank
  • keep exact in-memory route matching so global and context-scoped placeholders remain safe
  • add regression coverage for lookup arguments across global, context, and context-instance routes

Validation

  • uv run python -m unittest tests.test_runtime_environments
  • uv run --extra dev ruff check --fix --diff control_plane/product_config.py tests/test_runtime_environments.py
  • uv run --extra dev ruff check control_plane/product_config.py tests/test_runtime_environments.py
  • uv run --extra dev ruff format --check control_plane/product_config.py tests/test_runtime_environments.py
  • uv run --extra dev mypy control_plane/product_config.py tests/test_runtime_environments.py
  • uv run python -m unittest

@cbusillo cbusillo merged commit 74770f3 into main Jun 3, 2026
12 checks passed
@cbusillo cbusillo deleted the fix-runtime-placeholder-lookup-narrowing branch June 3, 2026 12:23
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