Add 7.1 migration guide

[christiangoerdes]

Summary by CodeRabbit

• Documentation
  • Added Membrane 7.1 migration guide with comprehensive examples and formatting improvements
  • Documented deprecations and behavior changes (MethodOverrideInterceptor removal, groovy interceptor adjustments, ApiKey extractor renames)
  • Described OpenAPI spec transition and updated configuration structure
  • Detailed new YAML syntax rules: inline list form, simplified headerFilter patterns, chain/sequence updates, and revised choose/otherwise ordering

[membrane-ci-server]

This pull request needs "/ok-to-test" from an authorized committer.

[coderabbitai]

📝 Walkthrough

Walkthrough

Added a "Migrate to Membrane 7.1" section to the migration guide documenting deprecations and YAML/OpenAPI configuration syntax changes (inline list form, headerFilter simplification, chainDef → chain, choose/otherwise semantics, OpenAPI renaming/relocation) and updated examples and formatting.

Changes

Cohort / File(s)	Summary
Migration Guide Updates docs/MIGRATION-GUIDE.md	Added "Migrate to Membrane 7.1" section detailing deprecations and new behaviours (removal of MethodOverrideInterceptor, groovy interceptor changes), YAML syntax enhancements (inline list elements, headerFilter inline patterns, chainDef → chain, choose/otherwise ordering), OpenAPI renaming/relocation (specs → openapi) and updated example code blocks and formatting.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• Migration guide for 7 #2620: Overlapping Migration Guide updates for Membrane 7 and YAML/OpenAPI migration notes.
• README.md YAML Security #2562: Related documentation changes migrating examples and YAML configuration semantics.

Suggested reviewers

• rrayst

Poem

"i'm a rabbit with a doc to write,
hopping through YAML by lantern light,
lists now short, and chains renamed,
OpenAPI's moved, the rules reclaimed.
nibble these notes — upgrade delight!" 🐇✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Add 7.1 migration guide' directly and clearly summarizes the main change—adding migration documentation for version 7.1, which matches the changeset's primary purpose.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 7-1-migration-guide

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

docs/MIGRATION-GUIDE.md (3)

239-239: Inconsistent indentation before "New:" label.

The "New:" label on line 239 has extra leading spaces compared to the "Old:" label on line 228, creating inconsistent formatting between the two examples.

📝 Proposed fix for consistent indentation

-New:
-  ```yaml
+New:
+```yaml

---

226-226: Consider clarifying the schema validation condition.

The parenthetical note "(Only when schema validation is deactivated)" may confuse users who don't know what schema validation refers to or how to deactivate it. Consider adding a brief explanation or a reference to where this setting is configured.

---

300-300: Clarify "case list" terminology.

The phrase "must be the last item of the case list" could be confusing since there isn't a distinct "case list" structure in the YAML. Based on the examples shown (lines 318-324), it would be clearer to say "must be the last item in the choose list" or "must appear after all case items."

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}