Skip to content

TEST: check-models gate demo (do not merge)#68

Closed
jimbobbennett wants to merge 3 commits into
mainfrom
test/check-models-gate-demo
Closed

TEST: check-models gate demo (do not merge)#68
jimbobbennett wants to merge 3 commits into
mainfrom
test/check-models-gate-demo

Conversation

@jimbobbennett

Copy link
Copy Markdown
Contributor

Temporary PR to demonstrate the check-models gate. Edits one line containing an outdated model in an airflow DAG; the other model lines in the file are untouched. Expect: 1 introduced (fails) + several pre-existing (warn). Will close + delete branch after.

jimbobbennett and others added 3 commits June 18, 2026 15:46
Skill + scanner that find outdated OpenAI / Anthropic / Google (Gemini)
model references and migrate them to current size-tier equivalents. The
CI gate scans only the lines a PR changes (paths: python typescript), so
it flags newly introduced outdated models and fails the check, without
blocking on pre-existing references. Notebook/code migrations land
separately as themed PRs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…sting

CI gate now scans the whole of each touched file and tags findings by whether
the PR changed the line — introduced fails, pre-existing (unchanged line of a
touched file) is a non-blocking warning. `python typescript` scan paths.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🤖 Model version check

✗ 1 outdated model reference(s) introduced — please update (this fails the check)

On lines this PR adds or changes:

Location Found Suggested Why
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:86 gpt-4o-mini gpt-5.4-mini mini tier → latest mini

⚠️ 6 pre-existing outdated model(s) in files this PR touches — not blocking

These are on unchanged lines, so the check still passes — but since you're already editing these files, consider updating them too.

Location Found Suggested Why
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:87 gpt-4o gpt-5.5 full tier → latest full
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:231 gpt-4o-mini gpt-5.4-mini mini tier → latest mini
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:249 gpt-4o-mini gpt-5.4-mini mini tier → latest mini
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:363 gpt-4o gpt-5.5 full tier → latest full
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:443 gpt-4o-mini gpt-5.4-mini mini tier → latest mini
python/cookbooks/airflow_example_dags/example_arize_ax_self_optimizing_loop_dag.py:524 gpt-4o-mini gpt-5.4-mini mini tier → latest mini

See the check-models skill. Policy date: 2026-06-18. Add check-models:ignore to a line to skip it.

ℹ️ Platform-wrapped IDs (Bedrock [region.]anthropic.claude-…, Databricks databricks-claude-…, OpenRouter/LiteLLM provider/model) are flagged on their embedded model name — bump the version but keep the platform's ID format (e.g. Bedrock 4.x needs a us./eu./apac. inference-profile prefix). See the skill's Platform-specific IDs section.

@jimbobbennett jimbobbennett deleted the test/check-models-gate-demo branch June 18, 2026 23:04
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