@phuc1403/musketeer 0.9.0 → 0.11.0

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Application/SeeThroughCases.cs

@@ -0,0 +1,89 @@
+namespace Sample.Application;
+
+// A method parameter of interface type is called: an edge to its implementation.
+public interface IDrawing
+{
+    string Title();
+}
+
+public sealed class TitleReading
+{
+    public string Read(IDrawing drawing) => drawing.Title();
+}
+
+// A call on a returned object: an edge to the implementations of what is returned.
+public interface IRecovery
+{
+    void Settle();
+}
+
+public sealed class RecoveryRouting(IEnumerable<IRecovery> recoveries)
+{
+    public IRecovery Find() => recoveries.First();
+}
+
+public sealed class CrashSettling(RecoveryRouting routes)
+{
+    public void Settle() => routes.Find().Settle();
+}
+
+// A record property that calls a component: reading that property couples the reader; reading another does not.
+public sealed record Job(int Id, string Callback)
+{
+    public string CallbackTarget => CallbackNormalizing.Normalize(Callback);
+}
+
+public static class CallbackNormalizing
+{
+    public static string Normalize(string url) => url.Trim();
+}
+
+public sealed class JobReporting
+{
+    public string Target(Job job) => job.CallbackTarget;
+}
+
+public sealed class JobCounting
+{
+    public int Count(Job job) => job.Id;
+}
+
+// A helper that calls a component: the helper's caller depends on the component; the helper is no node.
+public sealed class LabelFormat(RunClock clock)
+{
+    public string Stamp(string text) => $"{text} {clock.Elapsed()}";
+}
+
+public sealed class Labelling(LabelFormat format)
+{
+    public string Label(string text) => format.Stamp(text);
+}
+
+// Only constructing a helper runs its field initializers: they count.
+public sealed class StampedText
+{
+    private readonly string _stamp = PriceRules.Check(1m) ? "ok" : "no";
+
+    public string Apply(string text) => text + _stamp;
+}
+
+public sealed class Stamping
+{
+    public object Make() => new StampedText();
+}
+
+// Helpers calling each other in a cycle: the walk ends, and what they reach is reached once.
+public static class PingText
+{
+    public static string Ping(int n) => n > 0 ? PongText.Pong(n - 1) : 1m.AsMoney();
+}
+
+public static class PongText
+{
+    public static string Pong(int n) => PingText.Ping(n);
+}
+
+public sealed class Echoing
+{
+    public string Echo() => PingText.Ping(3);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Billing.Application/OrderRunning.cs

@@ -0,0 +1,7 @@
+namespace Sample.Billing.Application;
+
+// Same simple name as Sample.Application.OrderRunning: exercises the name-clash rule.
+public sealed class OrderRunning
+{
+    public decimal Total(IEnumerable<decimal> lines) => lines.Sum();
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Billing.Application/Sample.Billing.Application.csproj

@@ -0,0 +1,11 @@
+<Project Sdk="Microsoft.NET.Sdk">
+  <PropertyGroup>
+    <TargetFramework>net10.0</TargetFramework>
+    <Nullable>enable</Nullable>
+    <ImplicitUsings>enable</ImplicitUsings>
+  </PropertyGroup>
+  <ItemGroup>
+    <!-- A source file outside the repo root: exercises the out-of-repo declaration rule. -->
+    <Compile Include="../../../SharedOutsideRepo/SharedClock.cs" Link="SharedClock.cs" />
+  </ItemGroup>
+</Project>

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/ForwardedNotifier.cs

@@ -0,0 +1,8 @@
+using Sample.Application;
+
+namespace Sample.Infrastructure;
+
+public sealed class ForwardedNotifier(Mailer mailer) : INotifier
+{
+    public void Notify(string message) => mailer.Send(message);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/InfrastructureDependencyInjection.cs

@@ -0,0 +1,16 @@
+using Microsoft.Extensions.DependencyInjection;
+using Sample.Application;
+
+namespace Sample.Infrastructure;
+
+// The three registration styles seen in real code: instance, factory, forwarding.
+public static class InfrastructureDependencyInjection
+{
+    public static IServiceCollection AddSampleInfrastructure(this IServiceCollection services)
+    {
+        services.AddSingleton<IOrderQueue>(new OrderQueue());
+        services.AddSingleton(sp => new ForwardedNotifier(new Mailer()));
+        services.AddSingleton<INotifier>(sp => sp.GetRequiredService<ForwardedNotifier>());
+        return services;
+    }
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/Mailer.cs

@@ -0,0 +1,6 @@
+namespace Sample.Infrastructure;
+
+public sealed class Mailer
+{
+    public void Send(string message) => Console.WriteLine(message);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/OrderQueue.cs

@@ -0,0 +1,11 @@
+using Sample.Application;
+
+namespace Sample.Infrastructure;
+
+// Implements a port, and reaches QueueStore only from within Infrastructure.
+public sealed class OrderQueue : IOrderQueue
+{
+    private readonly QueueStore _store = new QueueStore();
+
+    public void Enqueue(OrderRequest request) => _store.Add(new QueueEntry(request.Id));
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/QueueEntry.cs

@@ -0,0 +1,4 @@
+namespace Sample.Infrastructure;
+
+// Data reached from Infrastructure: still not a component.
+public record QueueEntry(int OrderId);

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/QueueStore.cs

@@ -0,0 +1,8 @@
+namespace Sample.Infrastructure;
+
+public sealed class QueueStore
+{
+    private readonly List<QueueEntry> _entries = [];
+
+    public void Add(QueueEntry entry) => _entries.Add(entry);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/ReviewCases.cs

@@ -0,0 +1,35 @@
+using Sample.Application;
+
+namespace Sample.Infrastructure;
+
+public sealed class OrderRequestHandler : IHandler<OrderRequest>
+{
+    public void Handle(OrderRequest item)
+    {
+    }
+}
+
+// Implements a different construction of the same generic port: never reached from IHandler<OrderRequest>.
+public sealed class TextHandler : IHandler<string>
+{
+    public void Handle(string item)
+    {
+    }
+}
+
+// Open generic: implements every construction, so it is reached from IHandler<OrderRequest>.
+public sealed class LoggingHandler<T> : IHandler<T>
+{
+    public void Handle(T item) => Console.WriteLine(item);
+}
+
+// Implements the interface whose static member is called: still not reached by that call.
+public sealed class DefaultPricing : IPricingDefaults
+{
+    public decimal Rate() => 2m;
+}
+
+public sealed class DiskStorage : StorageBase
+{
+    public override void Save(string text) => File.WriteAllText("out.txt", text);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/ReviewFixCases.cs

@@ -0,0 +1,13 @@
+using Sample.Application;
+
+namespace Sample.Infrastructure;
+
+public sealed class PingHandling : ICommandHandler<Ping>
+{
+    public void Handle(Ping command) => Console.WriteLine(command.N);
+}
+
+public sealed class PongHandling : ICommandHandler<Pong>
+{
+    public void Handle(Pong command) => Console.WriteLine(command.N);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/Sample.Infrastructure.csproj

@@ -0,0 +1,13 @@
+<Project Sdk="Microsoft.NET.Sdk">
+  <PropertyGroup>
+    <TargetFramework>net10.0</TargetFramework>
+    <Nullable>enable</Nullable>
+    <ImplicitUsings>enable</ImplicitUsings>
+  </PropertyGroup>
+  <ItemGroup>
+    <PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="10.0.0" />
+  </ItemGroup>
+  <ItemGroup>
+    <ProjectReference Include="../Sample.Application/Sample.Application.csproj" />
+  </ItemGroup>
+</Project>

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/SeeThroughCases.cs

@@ -0,0 +1,18 @@
+using Sample.Application;
+
+namespace Sample.Infrastructure;
+
+public sealed class DwgDrawing : IDrawing
+{
+    public string Title() => "drawing";
+}
+
+public sealed class ExportRecovery : IRecovery
+{
+    public void Settle() => Console.WriteLine("export settled");
+}
+
+public sealed class ImportRecovery : IRecovery
+{
+    public void Settle() => Console.WriteLine("import settled");
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/src/Sample.Infrastructure/UnreachedPlumbing.cs

@@ -0,0 +1,7 @@
+namespace Sample.Infrastructure;
+
+// Nothing reaches this, yet it is a candidate: every class in scope is listed, reached or not.
+public sealed class UnreachedPlumbing
+{
+    public int Answer() => 42;
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/tests/Sample.Tests/FakeOrderQueue.cs

@@ -0,0 +1,11 @@
+using Sample.Application;
+
+namespace Sample.Tests;
+
+// Test double implementing a port. Test projects are never loaded, so it must never become an edge target.
+public sealed class FakeOrderQueue : IOrderQueue
+{
+    public List<OrderRequest> Queued { get; } = [];
+
+    public void Enqueue(OrderRequest request) => Queued.Add(request);
+}

package/template/.claude/skills/logical-components/scripts/fixtures/SampleApp/tests/Sample.Tests/Sample.Tests.csproj

@@ -0,0 +1,11 @@
+<Project Sdk="Microsoft.NET.Sdk">
+  <PropertyGroup>
+    <TargetFramework>net10.0</TargetFramework>
+    <Nullable>enable</Nullable>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <IsTestProject>true</IsTestProject>
+  </PropertyGroup>
+  <ItemGroup>
+    <ProjectReference Include="../../src/Sample.Application/Sample.Application.csproj" />
+  </ItemGroup>
+</Project>

package/template/.claude/skills/logical-components/scripts/fixtures/SharedOutsideRepo/SharedClock.cs

@@ -0,0 +1,7 @@
+namespace Sample.Billing.Application;
+
+// Linked into Sample.Billing.Application from outside the SampleApp repo root: it cannot be cited, so it is dropped.
+public sealed class SharedClock
+{
+    public DateTime Now() => DateTime.UtcNow;
+}

package/template/.claude/skills/logical-components/scripts/global.json

@@ -0,0 +1,5 @@
+{
+  "test": {
+    "runner": "Microsoft.Testing.Platform"
+  }
+}

package/template/.claude/skills/logical-components/scripts/smoke-sample.sh

@@ -0,0 +1,26 @@
+#!/usr/bin/env bash
+# Smoke check of the built CLI against the checked-in golden fixture (no LLM involved):
+# batches must plan the fixture, and render must reproduce fixtures/RenderGolden/expected.md exactly.
+# Usage: scripts/smoke-sample.sh   (from anywhere; exits non-zero on any difference)
+set -euo pipefail
+
+here="$(cd "$(dirname "$0")" && pwd)"
+golden="$here/fixtures/RenderGolden"
+work="$(mktemp -d)"
+trap 'rm -rf "$work"' EXIT
+
+dotnet build "$here/LogicalComponents" -c Release -nologo -v q >/dev/null
+lc=(dotnet "$here/LogicalComponents/bin/Release/net10.0/LogicalComponents.dll")
+
+"${lc[@]}" batches --components "$golden/components.json" --agents 9 --out "$work/batches"
+test -f "$work/batches/batch-01.json" && test -f "$work/batches/batch-02.json"
+
+"${lc[@]}" render --components "$golden/components.json" --resp "$golden/resp" --repo "$golden/repo" --out "$work/out.md"
+
+# Git may check expected.md out with CRLF; render always writes LF.
+if diff <(tr -d '\r' < "$golden/expected.md") "$work/out.md"; then
+  echo "smoke: OK"
+else
+  echo "smoke: render output differs from expected.md" >&2
+  exit 1
+fi

package/template/.claude/skills/research/SKILL.md

@@ -1,69 +1,69 @@
----
-name: research
-description: "Research technical solutions and analyze architectures by spawning parallel researcher agents that search the web and cross-reference findings. Use for technology evaluation, best-practices research, solution design, and scalability/security/maintainability analysis."
-license: MIT
-argument-hint: "[topic]"
-metadata:
-  author: Phuc Pham
-  version: "2.1.1"
----
-
-# Research
-
-## 1. Decompose
-
-Break the topic into independent sub-questions. Note recency requirements (how current
-the information must be) and any specific angles the user cares about (security, perf,
-cost, migration path) — these shape both the queries and the final sections.
-
-## 2. Gather (parallel agents)
-
-For each independent sub-question, dispatch a `researcher` agent via the Agent tool
-(`subagent_type: "researcher"`). Run independent agents in a single message so they execute
-concurrently. Scale the count to the topic's breadth; don't spawn overlapping agents.
-
-Give each agent **one** sub-question plus any recency/angle constraints from step 1. The
-agent handles query crafting, source prioritization, and returning findings with URLs.
-Keep synthesis (steps 3–4) in the main agent.
-
-## 3. Cross-reference
-
-Before synthesizing:
-- **Verify** each significant claim across multiple independent sources.
-- **Check dates** — flag stale information.
-- **Surface conflicts** explicitly rather than silently picking a side.
-- **Flag single-source claims** as unverified; discard or downgrade what fails.
-
-## 4. Report
-
-Save to the output path if the main agent provided one; otherwise return inline. Cite
-sources throughout. Sacrifice grammar for concision.
-
-Adapt the structure to the topic — don't pad with empty sections. A typical shape:
-
-```markdown
-# Research: [Topic]
-
-## Summary
-[Key findings + recommendation, 1-3 paragraphs]
-
-## Findings
-[Organized by sub-question. Add security / performance / cost subsections
- only where they're relevant to this topic.]
-
-## Conflicts & Caveats
-[Conflicting or single-source claims, stale info]
-
-## Recommendations
-[Actionable: what to do, code examples, pitfalls. Include a comparison if evaluating options.]
-
-## References
-[Linked sources]
-
-## Open Questions
-[Unresolved items — always include, even if empty.]
-```
-
-**Standards:** accuracy (cross-referenced), currency (prefer last 12 months unless history
-matters), always cite with links. Deliver strategic intelligence that enables a decision,
-not a pile of links.
+---
+name: research
+description: "Research technical solutions and analyze architectures by spawning parallel researcher agents that search the web and cross-reference findings. Use for technology evaluation, best-practices research, solution design, and scalability/security/maintainability analysis."
+license: MIT
+argument-hint: "[topic]"
+metadata:
+  author: Phuc Pham
+  version: "2.1.1"
+---
+
+# Research
+
+## 1. Decompose
+
+Break the topic into independent sub-questions. Note recency requirements (how current
+the information must be) and any specific angles the user cares about (security, perf,
+cost, migration path) — these shape both the queries and the final sections.
+
+## 2. Gather (parallel agents)
+
+For each independent sub-question, dispatch a `researcher` agent via the Agent tool
+(`subagent_type: "researcher"`). Run independent agents in a single message so they execute
+concurrently. Scale the count to the topic's breadth; don't spawn overlapping agents.
+
+Give each agent **one** sub-question plus any recency/angle constraints from step 1. The
+agent handles query crafting, source prioritization, and returning findings with URLs.
+Keep synthesis (steps 3–4) in the main agent.
+
+## 3. Cross-reference
+
+Before synthesizing:
+- **Verify** each significant claim across multiple independent sources.
+- **Check dates** — flag stale information.
+- **Surface conflicts** explicitly rather than silently picking a side.
+- **Flag single-source claims** as unverified; discard or downgrade what fails.
+
+## 4. Report
+
+Save to the output path if the main agent provided one; otherwise return inline. Cite
+sources throughout. Sacrifice grammar for concision.
+
+Adapt the structure to the topic — don't pad with empty sections. A typical shape:
+
+```markdown
+# Research: [Topic]
+
+## Summary
+[Key findings + recommendation, 1-3 paragraphs]
+
+## Findings
+[Organized by sub-question. Add security / performance / cost subsections
+ only where they're relevant to this topic.]
+
+## Conflicts & Caveats
+[Conflicting or single-source claims, stale info]
+
+## Recommendations
+[Actionable: what to do, code examples, pitfalls. Include a comparison if evaluating options.]
+
+## References
+[Linked sources]
+
+## Open Questions
+[Unresolved items — always include, even if empty.]
+```
+
+**Standards:** accuracy (cross-referenced), currency (prefer last 12 months unless history
+matters), always cite with links. Deliver strategic intelligence that enables a decision,
+not a pile of links.