Skip to content

refactor: strip premium prompts from migrate flow#510

Merged
laynepenney merged 2 commits intomainfrom
feat/533-strip-premium-migrate
Apr 9, 2026
Merged

refactor: strip premium prompts from migrate flow#510
laynepenney merged 2 commits intomainfrom
feat/533-strip-premium-migrate

Conversation

@laynepenney
Copy link
Copy Markdown
Collaborator

Summary

  • remove the interactive premium prompt from gr migrate
  • stop generating a premium-only CLAUDE.md section in the OSS migrate path
  • remove the premium field from migrate JSON output

Verification

  • cargo test migrate:: --lib
  • git diff --check

Copy link
Copy Markdown
Collaborator Author

@laynepenney laynepenney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA Review — grip#510

Clean, well-scoped cleanup. -26/+4 lines.

What's removed:

  • Interactive premium Confirm prompt from gr migrate
  • premium parameter from generate_claude_md()
  • Premium features block from generated CLAUDE.md
  • "premium" field from migrate JSON output

Test updated correctly — assertions flipped to !md.contains("Premium Features") and !md.contains("recall_identity").

No concerns. Rust compiler enforces no dangling callers of the removed parameter. Good first concrete artifact from the #533 audit.

Approve.

@laynepenney laynepenney marked this pull request as ready for review April 9, 2026 20:39
@laynepenney laynepenney merged commit 6f5b10e into main Apr 9, 2026
8 of 10 checks passed
@laynepenney laynepenney deleted the feat/533-strip-premium-migrate branch April 9, 2026 20:39
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