fix(revit): scope the PushButtonData release with a using on the IDisposable view

[thewrz]

This was written agentically; verify its assertions:

Why

#711 released PushButtonData in a finally block to satisfy cs/local-not-disposed on every leg; the rescan then raised code-quality finding #7 (cs/missed-using-statement), which prefers a using statement over manual disposal in finally. A plain using var on the PushButtonData cannot compile on the Revit 2024 leg, where the type is not IDisposable.

What

using var releaseButtonData = buttonData as IDisposable; right after construction: null (a no-op) on Revit 2024, a release at scope exit on 2025+, one code path for all legs, and the using form both queries accept. No #if, no finally.

Testing

• All three legs compile with the CI commands against the real Revit API reference assemblies (2024 / 2025 / 2027)
• CI green; code-quality finding feat(db): migration runner and core schema #7 closes on the next scan of main with no replacement
• Manual: ribbon tab + Health Check button still appear in Revit (Windows + Revit only)

🤖 Co-authored by Claude Fable 5.1.

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Team
• Run ID: afaf99b4-e692-40fa-84ea-acfe42370990

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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