Skip to content

Conversation

@ncipollina
Copy link
Contributor

Summary

Comprehensive documentation fixes addressing installation commands, broken links, and test command updates.

Changes

1. Fixed Installation Commands (Prerelease Flag)

Updated all installation commands to include --prerelease flag since the package is currently 1.0.0-beta:

  • README.md: Updated dotnet add package command
  • docs/getting-started/installation.md: Updated CLI, PowerShell, and .csproj examples
  • docs/getting-started/quick-start.md: Updated installation step
  • docs/index.md: Updated installation command

2. Removed Broken Troubleshooting Link

  • README.md: Removed link to non-existent troubleshooting documentation (created stub instead)

3. Created Stub Documentation Files

Created 5 new documentation files with "Coming Soon" placeholders and planned content outlines:

  • docs/usage/opt-out.md - Opt-out mechanisms for decorators
  • docs/advanced/keyed-services.md - Internal keyed services strategy explanation
  • docs/advanced/performance.md - Performance characteristics and benchmarks
  • docs/advanced/troubleshooting.md - Common issues and solutions
  • docs/api-reference/generated-code.md - Structure of generated interceptor code

All stubs include:

  • "Coming Soon" info boxes
  • Overview of planned content
  • Links to related documentation
  • Placeholder checklist for future content

4. Updated Test Commands for MTP

Updated test commands throughout documentation to use Microsoft Testing Platform (MTP) format:

  • docs/contributing.md:
    • Updated "Getting Started" section
    • Updated "Before Submitting" checklist
    • Updated "Running Tests" section with framework-specific commands
  • CLAUDE.md: Updated build commands section with MTP notes

Changed from:

dotnet test

To:

dotnet run --project test/LayeredCraft.DecoWeaver.Generator.Tests/LayeredCraft.DecoWeaver.Generator.Tests.csproj --framework net8.0

Added notes explaining that the test project targets multiple frameworks (net8.0, net9.0, net10.0) and requires --framework specification.

Impact

  • Users can now successfully install the prerelease package without errors
  • All navigation links in mkdocs.yml now resolve (no more 404s)
  • Contributors understand how to run tests correctly with MTP
  • Documentation is more complete and discoverable

Testing

  • Verified all installation commands work with --prerelease flag
  • Confirmed all stub files render correctly in mkdocs
  • Tested test command with --framework net8.0

🤖 Generated with Claude Code

ncipollina and others added 4 commits November 5, 2025 13:11
Fixed multiple documentation issues:

1. Added --prerelease flag to all installation commands
   - README.md
   - docs/getting-started/installation.md
   - docs/getting-started/quick-start.md
   - docs/index.md
   - Updated PowerShell command to use -Prerelease
   - Updated version references to 1.0.0-beta

2. Removed broken troubleshooting link from README.md
   - Link pointed to non-existent docs/advanced/troubleshooting.md

3. Created stub documentation files with TODO placeholders
   - docs/usage/opt-out.md - Opt-out mechanisms
   - docs/advanced/keyed-services.md - Keyed services strategy
   - docs/advanced/performance.md - Performance characteristics
   - docs/advanced/troubleshooting.md - Common issues and solutions
   - docs/api-reference/generated-code.md - Generated code structure

   All stubs include "Coming Soon" notices and planned content outlines

4. Updated test commands to use MTP format
   - docs/contributing.md - Multiple locations
   - CLAUDE.md - Build commands section
   - Changed from 'dotnet test' to 'dotnet run --framework net8.0'
   - Added notes about MTP and multi-framework targeting

This resolves all broken navigation links in mkdocs.yml and ensures
users can successfully install the prerelease package.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Updated package reference to use wildcard version (1.0.0-beta.*) and added
note explaining that GHA appends build numbers to the version prefix.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Changed description from 'full open generic support' to 'support for closed
generic registrations' to accurately reflect current implementation.

The generator currently intercepts closed generic registrations like:
  services.AddScoped(typeof(IRepository<>), typeof(SqlRepository<>))

Open generic support (intercepting AddScoped<IRepository<T>, SqlRepo<T>>)
is planned for a future phase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Added 5 new tags to improve package searchability:
- csharp: Common search term for C# specific packages
- decorators: Plural form of decorator pattern
- code-generation: Alternative term for source generation
- cross-cutting: Describes cross-cutting concerns use case
- performance: Highlights zero runtime overhead benefit

Total tags increased from 10 to 15 for better NuGet search coverage.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@ncipollina ncipollina merged commit 00e9433 into main Nov 5, 2025
3 checks passed
@ncipollina ncipollina deleted the docs/fix-installation-and-broken-links branch November 5, 2025 18:19
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.

2 participants