Skip to content

refactor!: remove unused interface delegation generator - #40

Merged
StevenTCramer merged 5 commits into
masterfrom
task/031-remove-the-unused-interface-delegation-generator
Nov 8, 2026
Merged

StevenTCramer merged 5 commits into
masterfrom
task/031-remove-the-unused-interface-delegation-generator

Conversation

@StevenTCramer

Copy link
Copy Markdown
Contributor

Summary

Removed the unused interface delegation generator, its handwritten attribute, and the test-console sample that existed only for that feature (data-service.cs, i-logger.cs, console-logger.cs). i-data-processor.cs and test-class.cs stay for the markdown-docs sample. The three diagnostic rows are gone from AnalyzerReleases.Unshipped.md and were not reassigned. The readme no longer documents the generator. documentation/overview.md, the readme prefix table, and the analyzer reference overview name the live ids: TW0001, TW0002, TW0003, and TW0007. kanban/done/** was not edited.

Review round 1 M1: source/Directory.Build.props PackageTags no longer includes Delegate or Interface Delegation. The tags are TimeWarp; Source Generator; SourceGenerators; Analyzers, which matches the analyzers and generators that remain. Round-1 merged.md marks M1 fixed. Round 2 has not run, so review/disposition.md is not written yet.

Task 026 is archived. ganda kanban archive 026 ran from origin-home, committed on task/026-create-interface-delegation-generator, and landed on master as 0e4b3f5. This branch does not contain that land. The claim on 026 remains.

./bin/dev build succeeded with 0 warnings and 0 errors. ./bin/dev test passed: the test console printed the kebab-case, multi-dot, and PascalCase messages, and the analyzer suite reported Passed 11, Failed 0. ganda repo audit passes all checks. ganda repo audit --fix wrote a gitignored .vscode/settings.json window title so the window-icon check passes. The post-fix self-install still fails with the known MSB3030 missing dev.xml; ./bin/dev build is the build that passed.

This fix pass re-ran the same gates after the tag edit. ./bin/dev build succeeded again with 0 warnings and 0 errors and packed artifacts/packages/TimeWarp.SourceGenerators.1.0.0-beta.11.nupkg. That package's nuspec <tags> are TimeWarp Source Generator SourceGenerators Analyzers. ./bin/dev test printed the same three console lines and reported Passed 11, Failed 0. ganda repo audit passed 36, failed 0, skipped 3. --fix was not needed.

Review disposition: clean. Two rounds, roster general (round 1 effort 2, round 2 effort 1). Final counts: bug 0, suggestion 1 fixed (M1, PackageTags), nit 0; 0 open, 0 wontfix. Round 2 re-ran ./bin/dev build (0 warnings, 0 errors) and ./bin/dev test (Passed 11, Failed 0). Artifacts: review/review-framework.md, review/round-2/merged.md, review/disposition.md.

Out of this repo, still describing the removed feature: hub profile/README.md and timewarp-software public/packages/timewarp.sourcegenerators/index.md.

Test plan

rg -n "InterfaceDelegationGenerator|ImplementsAttribute|\\[Implements\\]|TW0004|TW0005|TW0006" \
  source tests readme.md documentation

Kanban task 031
Session: Grok 01a1bc11-9829-7471-9676-451eab200c95

StevenTCramer and others added 5 commits November 8, 2026 23:49
Nothing in the org calls [Implements]. The generator, the handwritten
attribute, and the sample that existed only for it are gone. TW0004
through TW0006 stay retired.

why: task 031

Co-Authored-By: grok <noreply@x.ai>
PackageTags still listed Delegate and Interface Delegation after the
generator was deleted, so package search kept offering a feature this
package no longer ships. The tags are now TimeWarp, Source Generator,
SourceGenerators, and Analyzers. The task kitchen records review M1
as fixed and waits for the host re-review.

why: task 031 review M1

Co-Authored-By: grok <noreply@x.ai>
@StevenTCramer
StevenTCramer merged commit 4e0a854 into master Nov 8, 2026
1 check passed
@StevenTCramer
StevenTCramer deleted the task/031-remove-the-unused-interface-delegation-generator branch November 8, 2026 17:20
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