Add HandlerTemplate to ScanForTypesAttribute by Copilot · Pull Request #58 · Dreamescaper/ServiceScan.SourceGenerator · GitHub
Skip to content

Add HandlerTemplate to ScanForTypesAttribute#58

Merged
Dreamescaper merged 7 commits into
mainfrom
copilot/add-template-handler-per-type
Apr 11, 2026
Merged

Add HandlerTemplate to ScanForTypesAttribute#58
Dreamescaper merged 7 commits into
mainfrom
copilot/add-template-handler-per-type

Conversation

Copilot AI commented Apr 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

ScanForTypesAttribute lacked a way to generate per-type expressions beyond method invocations, forcing users to fall back to Activator when constructors require arguments.

Changes

  • New HandlerTemplate property on ScanForTypesAttribute: a string template where T (word-boundary matched) is replaced with the fully-qualified name of each found type
    • For collection-returning methods: template becomes each array element expression
    • For void / first-param-returning methods: template becomes a statement (; appended if absent)
    • Incompatible with Handler — new DI0016 diagnostic enforces this
  • MissingCustomHandlerOnGenerateServiceHandler (DI0013) no longer fires when HandlerTemplate is set
  • Template placeholder uses \bT\b regex (static compiled) for safe word-boundary replacement

Usage

// Collection return — generate instances with constructor args
[ScanForTypes(AssignableTo = typeof(BaseType), HandlerTemplate = "new T(argument)")]
public partial BaseType[] GetTypeInstances(string argument);

// Void method — generate statements
[ScanForTypes(AssignableTo = typeof(BaseType), HandlerTemplate = "registry.Add(new T(argument))")]
public partial void RegisterInstances(BaseTypeRegistry registry, string argument);

Copilot AI linked an issue Apr 9, 2026 that may be closed by this pull request
Add ability to set template of handler per each type #57
Closed
Copilot AI changed the title [WIP] Add ability to set template of handler per each type Add HandlerTemplate to ScanForTypesAttribute Apr 9, 2026
Copilot AI requested a review from Dreamescaper April 9, 2026 17:21
@Dreamescaper Dreamescaper marked this pull request as ready for review April 11, 2026 15:47
@Dreamescaper

Dreamescaper commented Apr 11, 2026

Copy link
Copy Markdown
Owner

Copilot AI commented Apr 11, 2026

Copy link
Copy Markdown
Contributor Author

@copilot
update README.md

Done in ea6369c. Added a new "Create instances using constructors with parameters" example section showing both collection-returning and void-method usage, and added HandlerTemplate to the ScanForTypes parameters table.

@Dreamescaper

Dreamescaper commented Apr 11, 2026

Copy link
Copy Markdown
Owner

@copilot
Make README info less about constructors specifically, and more about any non-standard cases, when Handler is not flexible enough. Constructors would be just one example. Static methods with multiple parameters, typeof manipulations etc.

Copilot AI commented Apr 11, 2026

Copy link
Copy Markdown
Contributor Author

Dreamescaper
Dreamescaper reviewed Apr 11, 2026
View reviewed changes
Comment thread README.md Outdated
Copilot AI requested a review from Dreamescaper April 11, 2026 17:50
Dreamescaper
Dreamescaper reviewed Apr 11, 2026
View reviewed changes
Comment thread README.md Outdated
Copilot AI requested a review from Dreamescaper April 11, 2026 17:53
@Dreamescaper Dreamescaper merged commit 8971c3b into main Apr 11, 2026
1 check passed
@Dreamescaper Dreamescaper deleted the copilot/add-template-handler-per-type branch April 11, 2026 17:56
This was referenced Apr 13, 2026
Closed
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Dreamescaper Dreamescaper Awaiting requested review from Dreamescaper

Assignees

@Dreamescaper Dreamescaper

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add ability to set template of handler per each type

2 participants

@Dreamescaper