Promote develop to main: template re-sync (CODESTYLE, tasks, CI smoke)

.editorconfig

@@ -36,7 +36,7 @@ indent_size = 2
 end_of_line = crlf
 indent_size = 2
 
-# Json files
+# Json and JsonC files
 [*.{json,jsonc}]
 end_of_line = crlf
 
@@ -54,6 +54,9 @@ end_of_line = crlf
 # C# files
 [*.cs]
 end_of_line = crlf
+# Suppressions follow CODESTYLE.md "Analyzer Diagnostics and Suppressions": prefer a
+# [SuppressMessage] attribute or the owning project's .editorconfig; relax a rule
+# repo-wide here only when it applies to every project (never a brownfield batch).
 dotnet_diagnostic.IDE0055.severity = none
 dotnet_analyzer_diagnostic.severity = suggestion
 csharp_indent_block_contents = true

.github/copilot-instructions.md

@@ -1,8 +1,12 @@
-# GitHub Copilot Instructions for LanguageTags
+# Copilot Instructions
+
+Repository conventions for GitHub Copilot (and any other AI agent reading this file).
 
 The **canonical guide is [AGENTS.md](../AGENTS.md)** at the repo root - read it first, including the [PR Review Etiquette](../AGENTS.md#pr-review-etiquette) review-loop contract this file's runbook implements. This file is intentionally narrow: commit/PR-title conventions (summarized inline so VS Code's commit-message and PR-title generators have them) plus the GitHub Copilot Review Runbook.
 
-For C# style rules, see [`CODESTYLE.md`](../CODESTYLE.md) at the repo root. Do not duplicate those rules here. **Project-specific conventions and API/behavioral contracts also belong in [AGENTS.md](../AGENTS.md), not here** - this file is intentionally limited to the inline commit/PR-title summary and the GitHub Copilot Review Runbook. Non-Copilot agents (Claude Code, Codex, Cursor, ...) are not directed to this file and don't read it by default, so any rule a reviewer must honor has to live in `AGENTS.md` to be provider-independent.
+For code-style rules, see [`CODESTYLE.md`](../CODESTYLE.md) at the repo root - one guide with a General section plus the .NET language section.
+
+Do not duplicate language-specific rules here. **Project-specific conventions and API/behavioral contracts also belong in [AGENTS.md](../AGENTS.md), not here** - this file is intentionally limited to the inline commit/PR-title summary and the GitHub Copilot Review Runbook. Non-Copilot agents (Claude Code, Codex, Cursor, ...) are not directed to this file and don't read it by default, so any rule a reviewer must honor has to live in `AGENTS.md` to be provider-independent.
 
 ## Commit Messages and Pull Request Titles
 
@@ -138,11 +142,13 @@ Issue-level Copilot comments (those in `issues/<N>/comments`) have no resolution
 Reply-body conventions:
 
 - Accepted bug/style fix: include fixing commit SHA and a one-line summary.
-- Declined style comment: cite the rule (AGENTS.md or CODESTYLE.md) and the existing-tree precedent.
+- Declined style comment: cite the rule (AGENTS.md or the CODESTYLE.md language section) and the existing-tree precedent.
 - Declined architecture proposal: one-sentence rationale.
 
 After the final push, sweep-resolve stale older threads for removed code paths.
 
 ## When in Doubt
 
-Read [AGENTS.md](../AGENTS.md) for this repo's conventions; [`CODESTYLE.md`](../CODESTYLE.md) is authoritative for C# style. Don't restate any of these files' rules in commit bodies or PR descriptions - keep those focused on the change itself. If you find a discrepancy that should be fixed in the template itself (this file or AGENTS.md is out of date, a rule is missing, something bit this repo and would bite the next derived repo), open an issue upstream in [`ptr727/ProjectTemplate`](https://github.com/ptr727/ProjectTemplate) rather than only fixing it locally - see [AGENTS.md "Staying in Sync with the Template"](../AGENTS.md#staying-in-sync-with-the-template).
+Read [AGENTS.md](../AGENTS.md) for this repo's conventions. For code-style rules, [`CODESTYLE.md`](../CODESTYLE.md) (its General section plus the relevant language section) is authoritative. Don't restate any of these files' rules in commit bodies or PR descriptions - keep those focused on the change itself.
+
+**In a derived repo:** if you find a discrepancy that should be fixed in the template itself (this file or AGENTS.md is out of date, a rule is missing, something bit this repo and would bite the next), open an issue upstream in [`ptr727/ProjectTemplate`](https://github.com/ptr727/ProjectTemplate) rather than only fixing it locally - see the template's [AGENTS.md "Staying in Sync and Reporting Drift Upstream"](https://github.com/ptr727/ProjectTemplate/blob/main/AGENTS.md#staying-in-sync-and-reporting-drift-upstream).

.github/workflows/build-nugetlibrary-task.yml

@@ -22,6 +22,12 @@ on:
       branch:
         required: true
         type: string
+      # Smoke mode: build for validation only and skip the artifact zip/upload. A PR smoke run has no consumer for
+      # the artifact (the github-release job is gated `!smoke`), so uploading it just burns artifact storage.
+      smoke:
+        required: false
+        type: boolean
+        default: false
     outputs:
       # Output of the uploaded artifact id
       artifact-id:
@@ -78,13 +84,16 @@ jobs:
             --skip-duplicate
 
       - name: Zip output step
+        if: ${{ !inputs.smoke }}
         run: |
           set -euo pipefail
           7z a -t7z ${{ runner.temp }}/${{ env.PROJECT_ARTIFACT }} ${{ runner.temp }}/publish/*
 
       # Branch-suffixed so the publisher's branch matrix can build both
       # branches in one run without colliding on the artifact name.
+      # Skipped on smoke: the github-release job is `!smoke`, so nothing would consume it.
       - name: Upload build artifacts step
+        if: ${{ !inputs.smoke }}
         id: artifact-upload-step
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
         with:

.github/workflows/build-release-task.yml

@@ -57,6 +57,8 @@ jobs:
       branch: ${{ inputs.branch }}
       # Push to NuGet.org, never on a smoke build.
       push: ${{ inputs.nuget && !inputs.smoke }}
+      # Skip the artifact zip/upload on smoke (nothing consumes it on a PR).
+      smoke: ${{ inputs.smoke }}
 
   github-release:
     name: Publish GitHub release job

.husky/task-runner.json

@@ -15,7 +15,7 @@
             ]
         },
         {
-            "name": ".Net Format",
+            "name": ".NET Format",
             "command": "dotnet",
             "args": [
                 "format",

.vscode/launch.json

@@ -5,7 +5,7 @@
             "name": "LanguageTagsCreate",
             "type": "coreclr",
             "request": "launch",
-            "preLaunchTask": ".Net Build",
+            "preLaunchTask": ".NET Build",
             "program": "${workspaceFolder}/.artifacts/bin/LanguageTagsCreate/debug/LanguageTagsCreate.dll",
             "args": [
                 "--codepath",

.vscode/tasks.json

@@ -1,8 +1,11 @@
 {
     "version": "2.0.0",
     "tasks": [
+        // .NET language group. A non-.NET repo drops this group and adds its own
+        // language's tasks. The first three tasks are the .NET clean-compile set
+        // (CODESTYLE.md) carried verbatim; the rest are convenience/project-specific.
         {
-            "label": ".Net Build",
+            "label": ".NET Build",
             "type": "process",
             "command": "dotnet",
             "args": [
@@ -20,7 +23,7 @@
             }
         },
         {
-            "label": ".Net Format",
+            "label": ".NET Format",
             "type": "process",
             "command": "dotnet",
             "args": [
@@ -37,9 +40,10 @@
                 "showReuseMessage": false,
                 "clear": false
             },
+            "dependsOrder": "sequence",
             "dependsOn": [
                 "CSharpier Format",
-                ".Net Build"
+                ".NET Build"
             ]
         },
         {
@@ -60,8 +64,9 @@
                 "clear": false
             }
         },
+        // Convenience / project-specific tasks (adapt or drop per repo).
         {
-            "label": ".Net Tool Update",
+            "label": ".NET Tool Update",
             "type": "process",
             "command": "dotnet",
             "args": [
@@ -94,7 +99,7 @@
             }
         },
         {
-            "label": ".Net Outdated Upgrade",
+            "label": ".NET Outdated Upgrade",
             "type": "process",
             "command": "dotnet",
             "args": [

AGENTS.md

@@ -138,7 +138,8 @@ Anti-pattern: don't keep flipping the code on the same style point. Flip the rul
 This repo is derived from [`ptr727/ProjectTemplate`](https://github.com/ptr727/ProjectTemplate) and re-syncs against it periodically, not just at creation.
 
 - **Verbatim carries.** Pull the current template version of each shared artifact and re-apply it, adapting only this repo's placeholders: [`.github/copilot-instructions.md`](./.github/copilot-instructions.md) (the Copilot review runbook - change only the `<owner>`/`<repo>`/`<N>` values in its API snippets), [`.markdownlint-cli2.jsonc`](./.markdownlint-cli2.jsonc), [`.editorconfig`](./.editorconfig), [`.gitattributes`](./.gitattributes), and this file's [PR Review Etiquette](#pr-review-etiquette) section. The `.editorconfig` EOL/per-extension block is always-verbatim; its `[*.cs]`/ReSharper block is .NET-only and is carried here. Keep `copilot-instructions.md` **narrow** (provider mechanics plus the commit/PR-title summary); project-specific conventions and API contracts live in this file (see [Library API Conventions](#library-api-conventions)), not there - non-Copilot agents are not directed to that file.
-- **CODESTYLE.md.** Keep [`CODESTYLE.md`](./CODESTYLE.md) as the full aggregate the template ships and re-sync the whole file, rather than hand-trimming per-language snippets.
+- **CODESTYLE.md.** Re-sync the whole file from the template, then keep the **General** section plus the **.NET** language section and drop the language sections this repo doesn't ship (the per-language sections are droppable, exactly like the `.editorconfig` `[*.cs]` block; this repo is .NET-only, so the Python section is dropped). Repo-root placement is load-bearing - `AGENTS.md` and `.github/copilot-instructions.md` link it by relative path. Adapt the in-section repo-specific bits: the .NET project-folder list, the `InternalsVisibleTo` project names, and the VS Code task labels. Replacing the file wholesale and dropping whole sections is simpler to keep current than hand-editing per-language snippets.
+- **.vscode/tasks.json.** Carry the named **clean-compile** task definitions verbatim - `.NET Build`, `CSharpier Format`, and `.NET Format` (which chains the first two then `dotnet format style --verify-no-changes`). Their names are owned by the `CODESTYLE.md` ".NET" section and their command sequence + arguments are the canonical clean-compile spec; don't loosen them. Convenience tasks (`.NET Tool Update`, `.NET Outdated Upgrade`, `Husky.Net Run`) are the adapt zone.
 - **Release notes.** Keep a short release-notes summary in [`README.md`](./README.md) and the full history in [`HISTORY.md`](./HISTORY.md); update both when cutting a release.
 - **Report drift upstream.** When a re-sync surfaces a template gap, an outdated instruction, or something that bit this repo and would bite the next derived repo, open an issue in [`ptr727/ProjectTemplate`](https://github.com/ptr727/ProjectTemplate) rather than only patching locally - the template is the single source of truth, and this upstream-issue rule is this repo's only cross-repo obligation. Do not maintain or reference a "known downstream" registry, and do not name sibling repositories in docs, comments, or workflows - that registry and the maintainer fan-out duty live in the template hub only.