Skip to content

Remove fintech/healthtech profiles and hide Config/Live tabs#57

Merged
fafaisland merged 1 commit into
mainfrom
claude/hopeful-thompson-777687
May 12, 2026
Merged

Remove fintech/healthtech profiles and hide Config/Live tabs#57
fafaisland merged 1 commit into
mainfrom
claude/hopeful-thompson-777687

Conversation

@fafaisland

@fafaisland fafaisland commented May 12, 2026

Copy link
Copy Markdown
Collaborator

Closes #56

Summary

  • Delete the bundled fintech and healthtech profile YAMLs (and the three CLI tests targeting them). The profile feature itself stays intact — the loader at engine_factory::load_scoring_config still reads any profiles/<name>.profile.yaml generically; only the example overlays are removed.
  • Replace remaining doc/comment references to those profile names with neutral placeholders so examples no longer point at deleted files.
  • Comment out the Config and Live tab buttons in the admin dashboard (crates/permit0-ui/static/index.html).
  • Comment out the cargo run -- calibrate test --profile my-profile example in CLAUDE.md since no profile ships in-tree anymore.

Why

The bundled profiles were illustrative, not production overlays — users are expected to author their own at profiles/<name>.profile.yaml. Shipping empty profiles/ makes the "create your own" workflow the obvious path and avoids implying the examples are blessed defaults. Hiding the Config and Live tabs takes those surfaces out of the dashboard until they're ready.

Test plan

  • cargo build --workspace clean
  • cargo build -p permit0-cli --tests clean
  • Open the admin dashboard and confirm Config / Live tabs are not visible
  • cargo nextest run --workspace green in CI

🤖 Generated with Claude Code

Drops the example profile YAMLs (`profiles/fintech.profile.yaml`,
`profiles/healthtech.profile.yaml`) and the three CLI tests that
targeted them. The profile feature stays — the loader still reads any
`profiles/<name>.profile.yaml` generically — only the bundled examples
are removed.

Doc/comment references to the deleted profile names are replaced with
neutral placeholders so examples don't point at files that no longer
exist.

Also disables (comments out) the dashboard "Config" and "Live" tab
buttons in the admin UI, and comments out the `--profile my-profile`
calibrate example in CLAUDE.md since no profile ships in-tree by
default.
@fafaisland fafaisland requested a review from AnissL93 as a code owner May 12, 2026 22:36
@fafaisland fafaisland merged commit 76b0960 into main May 12, 2026
8 checks passed
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.

[Developer Feedback] Profile is confusing

1 participant