Skip to content

feat: Migration toolkit gap detection Server.Transfer, HttpContext.Current.Session, ThreadAbortException#538

Open
csharpfritz wants to merge 1 commit intoFritzAndFriends:devfrom
csharpfritz:feature/toolkit-gap-detection
Open

feat: Migration toolkit gap detection Server.Transfer, HttpContext.Current.Session, ThreadAbortException#538
csharpfritz wants to merge 1 commit intoFritzAndFriends:devfrom
csharpfritz:feature/toolkit-gap-detection

Conversation

@csharpfritz
Copy link
Copy Markdown
Collaborator

Summary

Adds gap detection for Web Forms patterns that have no BWFC shim or require special handling during migration.

CLI Transform Enhancements

  • ServerShimTransform: Detect Server.Transfer/GetLastError/ClearError (no shim, manual rewrite TODOs)
  • SessionDetectTransform: Detect HttpContext.Current.Session[ replace with Session[, emit DI guidance
  • ResponseRedirectTransform: Detect catch(ThreadAbortException) dead code + endResponse=true warnings

Documentation Patches

  • bwfc-migration SKILL.md: Gap patterns section, non-page DI example
  • copilot-instructions-template.md: Gotchas 11-14
  • bwfc-data-migration SKILL.md: ThreadAbortException warning

Tests

  • 15 new unit tests (388 total, all passing)

…ion, ThreadAbortException

- ServerShimTransform: detect Server.Transfer(), GetLastError(), ClearError() with NO-SHIM TODOs
- SessionDetectTransform: detect HttpContext.Current.Session[ and replace with Session[
- ResponseRedirectTransform: detect catch(ThreadAbortException) dead code and endResponse=true warnings
- Added 15 new unit tests (388 total, all passing)
- Updated bwfc-migration SKILL.md with gap patterns section
- Updated copilot-instructions-template.md with gotchas 11-14
- Updated bwfc-data-migration SKILL.md with ThreadAbortException warning

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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