[WIKI-621]regression: file handler types

[iam-vipin]

Description

this fix ensuring that the intersection with TExtendedFileHandler is applied to the correct scope.

Type of Change

• Bug fix (non-breaking change which fixes an issue)

Screenshots and Media (if applicable)

Test Scenarios

References

Summary by CodeRabbit

• Refactor
  • Streamlined internal type configurations to improve consistency and compatibility with tooling.
  • Clarified typing boundaries to reduce ambiguity and improve future maintainability.
  • No changes to features, behavior, or UI; existing workflows continue to work as before.
  • Enhances developer ergonomics and reliability of type checks without impacting end-user functionality.

[coderabbitai]

Walkthrough

The pull request restructures the TFileHandler type in packages/editor/src/core/types/config.ts by moving the intersection with TExtendedFileHandler outside the object type, altering the placement of the type intersection without changing the listed properties.

Changes

Cohort / File(s)	Summary
Type declaration restructuring packages/editor/src/core/types/config.ts	Adjusted TFileHandler to apply & TExtendedFileHandler after the object type’s closing brace; no property additions or removals within TFileHandler’s object shape.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

A nibble of types, a hop through the code,
I shuffle the braces where intersections load.
No fields have wandered, no props gone astray—
Just tidier trails on a sunny dev day.
Thump-thump, approve—then off I go! 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description Check	⚠️ Warning	The description includes the required Description and Type of Change sections but leaves the Test Scenarios section unpopulated and retains placeholder comments in the Screenshots and References sections, meaning it does not document how the fix was verified or clarify whether screenshots or related issues exist.	Please provide detailed test scenarios demonstrating how you verified that the intersection with TExtendedFileHandler now applies correctly, and either populate or remove the placeholder comments in the Screenshots and References sections to complete the template.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.
Title Check	✅ Passed	The pull request title “[WIKI-621]regression: file handler types” directly signals that the change addresses a regression in the file handler type definitions, which aligns with the primary modification in the changeset. It is concise, focused on the key area of impact, and includes the project’s ticket identifier without introducing unnecessary detail.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-config_types

---

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.}

[Copilot]

Pull Request Overview

This PR fixes a TypeScript type definition issue by moving the intersection with TExtendedFileHandler to the correct scope in the TFileHandler type definition.

• Corrected placement of the intersection operator for TExtendedFileHandler

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[makeplane]

Linked to Plane Work Item(s)

• [WIKI-621] Feat: Drawio integration

This comment was auto-generated by Plane