Improve GitRepoPublisher and RunClaude script

- Use ParametricString in GitRepoPublisher for URL and commit message to support $(PackageVersion)
- Rename RunClaude.g.ps1 to RunClaude.ps1 and add Docker environment check
- Add important notice to CLAUDE.md about reading skills before work

Author: gfraiteur

CLAUDE.md

@@ -2,6 +2,9 @@
 
 Build orchestration SDK for PostSharp/Metalama repositories.
 
+> [!IMPORTANT]
+> Read this file entirely before doing any work or answering any question for this project, especially regarding loading skills.
+
 ## Discovering Plugin Skills
 
 The `postsharp-engineering` plugin provides skills and slash commands. To discover them:

src/PostSharp.Engineering.BuildTools/Build/Publishing/GitRepoPublisher.cs

@@ -17,16 +17,16 @@ namespace PostSharp.Engineering.BuildTools.Build.Publishing;
 [PublicAPI]
 public class GitRepoPublisher : ArtifactPublisher
 {
-    private readonly string _gitHubUrl;
-    private readonly string _commitMessage;
+    private readonly ParametricString _gitHubUrl;
+    private readonly ParametricString _commitMessage;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="GitRepoPublisher"/> class.
     /// </summary>
     /// <param name="files">The pattern matching the zip file(s) to publish.</param>
-    /// <param name="gitHubUrl">The GitHub repository URL to publish to.</param>
-    /// <param name="commitMessage">The commit message to use when pushing changes.</param>
-    public GitRepoPublisher( Pattern files, string gitHubUrl, string commitMessage ) : base( files )
+    /// <param name="gitHubUrl">The GitHub repository URL to publish to. Supports parameters like <c>$(PackageVersion)</c>.</param>
+    /// <param name="commitMessage">The commit message to use when pushing changes. Supports parameters like <c>$(PackageVersion)</c>.</param>
+    public GitRepoPublisher( Pattern files, ParametricString gitHubUrl, ParametricString commitMessage ) : base( files )
     {
         this._gitHubUrl = gitHubUrl;
         this._commitMessage = commitMessage;
@@ -43,13 +43,13 @@ public override SuccessCode PublishFile(
 
         console.WriteMessage( $"Publishing '{file}' to git repository '{this._gitHubUrl}'." );
 
-        // Expand environment variables in the URL and commit message.
-        var gitHubUrl = Environment.ExpandEnvironmentVariables( this._gitHubUrl );
-        var commitMessage = Environment.ExpandEnvironmentVariables( this._commitMessage );
+        // Expand parametric strings like $(PackageVersion).
+        var gitHubUrl = this._gitHubUrl.ToString( buildArguments );
+        var commitMessage = this._commitMessage.ToString( buildArguments );
 
         if ( string.IsNullOrEmpty( gitHubUrl ) )
         {
-            console.WriteError( "The GitHub URL is empty or the environment variable is not defined." );
+            console.WriteError( "The GitHub URL is empty." );
 
             return SuccessCode.Fatal;
         }

src/PostSharp.Engineering.BuildTools/Resources/DockerBuild.ps1

@@ -559,13 +559,13 @@ if (-not $BuildImage)
         {
             # Non-interactive mode with prompt - no -it flags
             $dockerArgs = @()
-            $inlineScript = "${substCommandsInline}& c:\Init.g.ps1; ${copyClaudeJsonScript}cd '$SourceDirName'; & .\eng\RunClaude.g.ps1 -Prompt `"$ClaudePrompt`""
+            $inlineScript = "${substCommandsInline}& c:\Init.g.ps1; ${copyClaudeJsonScript}cd '$SourceDirName'; & .\eng\RunClaude.ps1 -Prompt `"$ClaudePrompt`""
         }
         else
         {
             # Interactive mode - requires TTY
             $dockerArgs = @("-it")
-            $inlineScript = "${substCommandsInline}& c:\Init.g.ps1; ${copyClaudeJsonScript}cd '$SourceDirName'; & .\eng\RunClaude.g.ps1"
+            $inlineScript = "${substCommandsInline}& c:\Init.g.ps1; ${copyClaudeJsonScript}cd '$SourceDirName'; & .\eng\RunClaude.ps1"
         }
 
         $dockerArgsAsString = $dockerArgs -join " "

…ing.BuildTools/Resources/RunClaude.g.ps1 …ering.BuildTools/Resources/RunClaude.ps1src/PostSharp.Engineering.BuildTools/Resources/RunClaude.g.ps1 renamed to src/PostSharp.Engineering.BuildTools/Resources/RunClaude.ps1 src/PostSharp.Engineering.BuildTools/Resources/RunClaude.g.ps1 renamed to src/PostSharp.Engineering.BuildTools/Resources/RunClaude.ps1

@@ -7,6 +7,12 @@ param(
 
 $ErrorActionPreference = "Stop"
 
+if ($env:RUNNING_IN_DOCKER -ne "true")
+{
+    Write-Error "This script must be run inside a Docker container. Set RUNNING_IN_DOCKER=true to override."
+    exit 1
+}
+
 Write-Host "Starting Claude CLI..." -ForegroundColor Green
 
 # Run Claude