• Migrate resource generation tool from Python to C#
• Replace py_binary with csharp_binary in Bazel build
• Implement GenerateResourcesTool.cs with argument parsing
• Remove legacy Python script and update documentation

New C# resource generation tool implementation

• New C# console tool that generates ResourceUtilities partial class
• Parses --output and --input arguments with IDENT=path format
• Reads JS files and embeds them as C# raw string literals with 5-quotes
• Creates output directory if needed and writes UTF-8 encoded file

dotnet/private/GenerateResourcesTool.cs

Update documentation for C# migration

• Update docstring to reflect C# tool instead of Python script
• Change description from "via a Python tool" to generic implementation

dotnet/private/generate_resources.bzl

Remove legacy Python resource generation tool

• Remove entire Python implementation (87 lines)
• Eliminates Python-specific argument parsing and file handling logic

dotnet/private/generate_resources_tool.py

Description

--input values are only superficially parsed and then directly used to emit C# ({name} and a
fixed 5-quote raw string delimiter), so invalid identifiers or JS content containing """"" can
cause later compiler failures instead of a clear upfront error.

Code

dotnet/private/GenerateResourcesTool.cs[R57-65]

+foreach (var (name, path) in inputs)
+{
+    var content = File.ReadAllText(path, Encoding.UTF8);
+    // Use a C# raw string literal with five quotes.
+    // The content must start on a new line and the closing quotes
+    // must be on their own line as well.
+    sb.AppendLine($"    internal const string {name} = {new string('"', 5)}");
+    sb.AppendLine(content);
+    sb.AppendLine($"{new string('"', 5)};");

Evidence

PR Compliance ID 12 requires validating configuration/external inputs early with clear, actionable
exceptions. The tool trims and stores Name/Path without validating non-empty/identifier legality
and then uses name verbatim in generated code while always emitting a 5-quote raw string
delimiter, which can break code generation/compilation with unclear downstream errors.

dotnet/private/GenerateResourcesTool.cs[28-37]
dotnet/private/GenerateResourcesTool.cs[57-65]
Best Practice: Learned patterns

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The resource generator accepts `--input IDENT=path` but does not validate that `IDENT` is a legal C# identifier (or even non-empty after trimming), and it always emits a 5-quote raw string delimiter regardless of file content. This can produce invalid generated C# and fail later with unclear errors.

## Issue Context
This tool is invoked by Bazel (`generated_resource_utilities`), so failures should be fast and actionable (clear stderr message + nonzero exit) when inputs are invalid.

## Fix Focus Areas
- dotnet/private/GenerateResourcesTool.cs[28-37]
- dotnet/private/GenerateResourcesTool.cs[57-65]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Description

GenerateResourcesTool uses StringBuilder.AppendLine(content), which always appends an extra newline
after each resource, so ResourceUtilities.* constants will not match the input file contents
exactly. These constants are executed/parsed at runtime (JS evaluation and JSON parsing), so this
silent content drift can change behavior and makes byte-for-byte resource embedding impossible.

Code

dotnet/private/GenerateResourcesTool.cs[R59-65]

+    var content = File.ReadAllText(path, Encoding.UTF8);
+    // Use a C# raw string literal with five quotes.
+    // The content must start on a new line and the closing quotes
+    // must be on their own line as well.
+    sb.AppendLine($"    internal const string {name} = {new string('"', 5)}");
+    sb.AppendLine(content);
+    sb.AppendLine($"{new string('"', 5)};");

Evidence

The generator reads each file and then emits it into a raw string literal using AppendLine(content),
which guarantees at least one extra newline beyond the file’s own trailing newline (if any). The
generated constants are then used as executable JavaScript and parsed JSON at runtime, meaning the
embedded string content matters.

dotnet/private/GenerateResourcesTool.cs[57-66]
dotnet/src/webdriver/JavaScriptEngine.cs[136-144]
dotnet/src/webdriver/Firefox/FirefoxProfile.cs[292-300]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`dotnet/private/GenerateResourcesTool.cs` uses `sb.AppendLine(content)` when emitting raw string literals. `AppendLine` always appends an additional newline after `content`, so every embedded resource gets an extra trailing newline (and resources that already end with a newline will effectively gain a blank line).

### Issue Context
These embedded strings are used at runtime as JavaScript sources and as JSON content; the generator should ideally preserve the resource text exactly (or at least avoid adding extra, unintended newlines).

### Fix Focus Areas
- dotnet/private/GenerateResourcesTool.cs[57-66]

### Suggested approach
- Replace `sb.AppendLine(content);` with `sb.Append(content);`
- Ensure the raw string closing delimiter still starts on a new line:
 - If `content` does **not** end with `\n` (or `\r\n`), append a single newline once (e.g., `sb.AppendLine();`).
 - If it already ends with a newline, do not add another.
- Keep the closing delimiter emission unchanged.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools