update to use csproj if present, fallback to generated if not.

Author: semgrep-jamie

Dockerfile

@@ -1,21 +1,17 @@
-# Build on the official .NET SDK image (Debian-based)
-# Using 8.0 LTS for stability and long-term support
 FROM mcr.microsoft.com/dotnet/sdk:8.0
 
-# Install Node.js (required for the script)
 RUN apt-get update && apt-get install -y --no-install-recommends curl \
     && curl -fsSL https://deb.nodesource.com/setup_20.x | bash - \
     && apt-get install -y --no-install-recommends nodejs \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/*
 
-# Create a workspace and copy entrypoint script
+COPY convert-nuget.js package.json package-lock.json /tool/
 WORKDIR /tool
-COPY convert-nuget.js /tool/convert-nuget.js
+RUN npm install --production && chmod +x convert-nuget.js
 
-RUN chmod +x /tool/convert-nuget.js
-
-# Default entrypoint: process current working directory
+# Default working directory (can be overridden with -w)
 # The working directory should be mounted as a volume when running:
 # docker run -v "$(pwd):/workspace" -w /workspace <image> [options]
+WORKDIR /workspace
 ENTRYPOINT ["node", "/tool/convert-nuget.js"]

README.md

@@ -5,7 +5,7 @@ Recursively searches for `packages.config` files and converts each to `packages.
 ## Manual Usage
 
 ```bash
-node convert-nuget.js [--tfm <TFM>] [--root <DIR>]
+node convert-nuget.js [--tfm <TFM>] [--root <DIR>] [--fail-on-skipped]
 
 # Example: scan from current directory
 node convert-nuget.js
@@ -15,6 +15,9 @@ node convert-nuget.js --tfm net48
 
 # Example: scan from a specific directory
 node convert-nuget.js --root ./projects
+
+# Example: fail if any packages are skipped due to incompatibility
+node convert-nuget.js --fail-on-skipped
 ```
 
 ## Local Docker Usage
@@ -26,13 +29,14 @@ Pull the Docker image from GitHub Container Registry and run it locally:
 docker pull ghcr.io/semgrep/convert-nuget
 
 # Run the conversion (scan from current directory)
-docker run --rm -v $(pwd):/workspace -w /workspace ghcr.io/semgrep/convert-nuget
+# Note: -w /workspace is optional since /workspace is the default working directory
+docker run --rm -v $(pwd):/workspace ghcr.io/semgrep/convert-nuget
 
 # Run the conversion (scan from a specific subdirectory)
-docker run --rm -v $(pwd):/workspace -w /workspace ghcr.io/semgrep/convert-nuget --root /workspace/some/subdirectory
+docker run --rm -v $(pwd):/workspace ghcr.io/semgrep/convert-nuget --root /workspace/some/subdirectory
 
 # Run with custom target framework
-docker run --rm -v $(pwd):/workspace -w /workspace ghcr.io/semgrep/convert-nuget --tfm net48
+docker run --rm -v $(pwd):/workspace ghcr.io/semgrep/convert-nuget --tfm net48
 ```
 
 ## Docker CI Usage
@@ -41,17 +45,17 @@ docker run --rm -v $(pwd):/workspace -w /workspace ghcr.io/semgrep/convert-nuget
 # GitHub Actions example
 - name: Convert packages.config
   run: |
-    docker run -v ${{ github.workspace }}:/workspace -w /workspace ghcr.io/semgrep/convert-nuget
+    docker run -v ${{ github.workspace }}:/workspace ghcr.io/semgrep/convert-nuget
 
 # GitLab CI example
 convert-nuget:
   script:
-    - docker run -v $PWD:/workspace -w /workspace ghcr.io/semgrep/convert-nuget
+    - docker run -v $PWD:/workspace ghcr.io/semgrep/convert-nuget
 ```
 
 ```bash
-# Command line
-docker run -v $(pwd):/workspace -w /workspace ghcr.io/semgrep/convert-nuget
+# Command line (note: -w /workspace is optional)
+docker run -v $(pwd):/workspace ghcr.io/semgrep/convert-nuget
 ```
 
 ## GitHub Actions Usage
@@ -79,7 +83,7 @@ jobs:
 
       - name: Convert packages.config
         run: |
-          docker run -v "${{ github.workspace }}:/workspace" -w /workspace ghcr.io/semgrep/convert-nuget
+          docker run -v "${{ github.workspace }}:/workspace" ghcr.io/semgrep/convert-nuget
 
       - name: Commit updated lock files (if any)
         run: |
@@ -108,4 +112,5 @@ jobs:
 
 - `--tfm <TFM>`: Target framework moniker (default: `net472`)
 - `--root <DIR>`: Root directory to search (default: current working directory)
+- `--fail-on-skipped`: Exit with error code 2 if any packages are skipped due to incompatibility (default: warnings only)
 - `-h, --help`: Show help message