chore(scripts): delete one-shot scripts, the fix_tool crate, and unused workspace deps - #169
Merged
eric8810 merged 1 commit intoSep 12, 2026
Conversation
…ed workspace deps Closes arcships#168. Every file under scripts/ was audited for references (CI, docs, other scripts, source comments), for inputs that still exist, and for whether it can run at all. Sixteen scripts are one-shot migrations that already ran: the fix_profiles.rs series targets a `fix-profiles` package that does not exist; the four inventory editors flip cells in rfc/0004 for the provider-generation session; the two migrate_openai_compatible_* scripts removed `Shell` types that are already gone; gen_provider_registry.py reads a file that no longer exists; gen_responses_convert.py refuses to run by its own docstring; generate_all_providers.py, update_lib_rs.py and gen_vertex_maas_providers.py scaffolded files that have since been edited by hand and carry no generated marker, so re-running them would clobber those edits. scripts/fix_tool was a `Hello, world!` workspace member compiled on every workspace build. schemars, proc-macro2, syn and quote were declared as workspace dependencies and consumed by no crate. provider.rs no longer points at the deleted registry generator; the registry JSON is edited by hand. The root .gitignore now covers __pycache__/ everywhere instead of only under the Python binding. Kept on purpose: generate_thin_wrapper_cassettes.py (cassette provenance cited by conformance_test.rs), the two cassette converters, the two LiteLLM survey scripts, and responses_similarity_audit.py.
Contributor
Author
|
Withdrawn for now: the scope (which scripts to delete versus keep) is still under discussion in #168. Branch kept; will reopen or replace once settled. |
Contributor
Author
|
Reopened after settling the scope in #168: the deletion list is unchanged; the replacement for the removed generators is the registry maintenance flow (weekly models.dev diff report + monthly reachability probe, generator rule: do-not-edit header + CI |
This was referenced Sep 4, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #168. Track A, item A1 of #166.
Every file under
scripts/was audited for references (CI, docs, other scripts, source comments), for inputs that still exist, and for whether it can run at all. The issue carries the per-script evidence; the short version:Deleted (16 scripts, one crate, four workspace deps; −2,117 lines)
fix_profiles{,2..5}.rs: target afix-profilespackage that does not exist; their edit is already in the tree.check_missing.py,list_wrong_providers.py,revert_wrong_inventory.py,update_inventory_status.py: flipped cells inrfc/0004during the provider-generation session.migrate_openai_compatible_{test,standalone}.py: removedShelltypes that are already gone.gen_provider_registry.py: readsopenai_compat_registry.rs, which no longer exists.gen_responses_convert.py: refuses to run by its own docstring;responses_convert.rsis hand-maintained.generate_all_providers.py,update_lib_rs.py,gen_vertex_maas_providers.py: scaffolded the pre-phase-4 thin-wrapper.rsfiles once. Since the registry landed (28fe708) providers are added as rows inprovider_registry.json(e.g. InferenceHub, feat: add InferenceHub provider to OpenAI-compatible registry #22), the outputs carry no generated marker and have been hand-edited, so re-running would overwrite them. Restorable from history if ever needed.scripts/fix_tool: aHello, world!workspace member compiled on every build.schemars,proc-macro2,syn,quote: workspace dependencies with no consumer.Kept:
generate_thin_wrapper_cassettes.py(cassette provenance cited byconformance_test.rs; retires with B2), the two cassette converters, the two LiteLLM survey scripts,responses_similarity_audit.py(retires with B6), and all live tooling.Also:
provider.rsno longer points at the deleted generator; root.gitignorecovers__pycache__/everywhere.Note: RFC-0033 (which records this audit in its A1 row) is not part of this diff; it lands with #176, which also ships the unified sync tool that replaces the two LiteLLM scanners kept here.
Verification:
cargo check -p aimux-providers,cargo fmt --check; nothing in CI referenced a deleted file.