Skip to content

fix(import): reject invalid whitespace-only or unknown message roles#297

Open
694410194 wants to merge 1 commit into
CortexLM:mainfrom
694410194:codex/reject-invalid-import-roles
Open

fix(import): reject invalid whitespace-only or unknown message roles#297
694410194 wants to merge 1 commit into
CortexLM:mainfrom
694410194:codex/reject-invalid-import-roles

Conversation

@694410194
Copy link
Copy Markdown

Summary

  • reject empty and whitespace-only export message roles before import mutates message content
  • reject unknown roles instead of silently prefixing the imported text with bracketed role labels
  • add focused unit coverage for empty, whitespace-only, and unknown roles

Verification

  • git diff --check
  • rustc --version (not available in this environment, so cargo-based tests could not be run here)

Fixes #53211
/claim #53211

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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