Skip to content

[Repo Assist] Split HtmlDocument operations out of HtmlOperations.fs#1683

Merged
dsyme merged 2 commits intomainfrom
repo-assist/split-htmloperations-htmldocument-969a88c6e81d98f8
Mar 1, 2026
Merged

[Repo Assist] Split HtmlDocument operations out of HtmlOperations.fs#1683
dsyme merged 2 commits intomainfrom
repo-assist/split-htmloperations-htmldocument-969a88c6e81d98f8

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Mar 1, 2026

🤖 This PR was created by Repo Assist, an automated AI assistant.

Part of #1677 — splitting large files in the codebase.

What's changed

HtmlOperations.fs was 1169 lines and contained two conceptually separate concerns:

  • HtmlAttribute and HtmlNode operations (with their C# extension wrappers)
  • HtmlDocument operations (with its C# extension wrappers)

This PR splits the HtmlDocument module, HtmlDocumentExtensions type, and HtmlExtensions module into a new file HtmlDocumentOperations.fs, placed immediately after HtmlOperations.fs in the project.

Result:

  • HtmlOperations.fs: 1169 → 865 lines (HtmlAttribute + HtmlNode operations)
  • HtmlDocumentOperations.fs: new, 316 lines (HtmlDocument operations)

This follows the existing pattern in the codebase where HtmlNode.fs holds the HtmlNode type definition and HtmlOperations.fs holds operations on it — now HtmlDocumentOperations.fs holds operations on HtmlDocument.

Note: This issue should remain open per the maintainer's request. Not using "Fixes" or "Closes" for issue #1677.

Test Status

  • ✅ Build: succeeded (0 errors)
  • ✅ All tests passed: 489 design-time, 283 integration, 26 C# interop, 2 reference (total 800+)
  • ✅ Fantomas format check: passed

Generated by Repo Assist

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@afb00b92a9514fee9a14c583f059a03d05738f70

Generated by Repo Assist for issue #1677

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@afb00b92a9514fee9a14c583f059a03d05738f70

…entOperations.fs

HtmlOperations.fs (1169 lines) contained both HtmlNode/HtmlAttribute operations
and HtmlDocument operations. Split HtmlDocument module, HtmlDocumentExtensions,
and HtmlExtensions into a new HtmlDocumentOperations.fs file, reducing
HtmlOperations.fs to 865 lines.

Part of #1677 (splitting large files).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dsyme dsyme marked this pull request as ready for review March 1, 2026 13:38
@dsyme dsyme merged commit c87020d into main Mar 1, 2026
2 checks passed
@dsyme dsyme deleted the repo-assist/split-htmloperations-htmldocument-969a88c6e81d98f8 branch March 1, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Split large files in this codebase

1 participant