Re-sync from template #167: docs/comment cleanup, CI storage limits, new tasks

[ptr727]

Upstream ptr727/ProjectTemplate#167 landed on develop (closes upstream #157-#164). Heads-up to re-sync this repo. It is not yet promoted to main, so expect possible follow-ups as downstreams apply these.

Re-sync the verbatim carries

• .github/copilot-instructions.md - the duplicated PR-title block and historic/defensive narrative were removed; the Copilot review runbook mechanics are unchanged. Copy the new version and re-adapt <owner>/<repo>/<N>.
• .editorconfig - the [*.cs]/ReSharper block is now marked .NET-only; the EOL/per-extension block stays always-verbatim. This repo ships .NET, so keep the [*.cs]/ReSharper block.
• If this repo is adopting .gitattributes (* -text) for the first time, do the one-time line-ending normalization (convert existing files to their .editorconfig ending in a dedicated commit).

Adopt the AGENTS.md rule updates (adapt to this repo's AGENTS)

• Comments house-rule: concise, current-state, no cross-project references (don't name sibling repos), no rule citations, match ~120-col. Applies to code and workflow comments.
• Markdown: trailing backslash (\) is blessed for intentional hard line breaks, preferred over trailing whitespace.
• Versioning: drop any "keep develop ahead of main" framing; bump version.json only for functional changes (NBGV advances the patch automatically).
• CI storage: set retention-days: 1 on intermediate actions/upload-artifact steps (default 90d piles up against the 2 GB account quota); cache Docker layers to a registry tag (type=registry), not the GitHub Actions cache (type=gha).
• Patterns: keep a README release-notes summary + full HISTORY.md; keep CODESTYLE.md as the full aggregate and re-sync the whole file; the "name sibling repos" prohibition + the upstream-issue rule are the only cross-repo obligations (the Known Downstream registry stays in the template hub only).

Workflows

• Re-sync the trimmed comments in the carried release/*-task.yml workflows (only comments changed upstream; logic, CRLF, and SHA pins preserved), and add retention-days: 1 to intermediate artifact uploads.

Reference

Template develop @ 212d8a8; full diff in the PR above.

[ptr727]

Template update — current state (2026-06-22)

Three more upstream changes landed on develop since #176 (CODESTYLE/tasks at f526b55, workflow fix at 402e862; not yet on main). Re-sync alongside the items above.

1. AGENTS.md + Copilot runbook (ProjectTemplate#173)

• Confirms project-specific conventions / API contracts belong in AGENTS.md (you already moved your "Library API Conventions" there — that move is the resolution this PR documents). .github/copilot-instructions.md stays narrow.
• The Copilot review runbook is hardened: a review with no inline comments is a completed clean pass (don't re-trigger/escalate); round 1 is auto-seeded — poll for it before self-triggering; read the bot node id from the formal review (or, if only an issue comment exists, the comment author via GraphQL ... on Bot { id }); UI-seed only when polling shows no formal review. Re-sync .github/copilot-instructions.md verbatim.

2. Single consolidated CODESTYLE.md + carry contract (#175, #178)

• CODESTYLE.md is now one file (General → .NET → Python), each language section droppable like .editorconfig [*.cs]. CODESTYLE.md and .vscode/tasks.json are now verbatim-carry artifacts; root placement is load-bearing. Re-sync your CODESTYLE.md to the consolidated file, keeping General + .NET and dropping the Python section.
• New General rules: official-tooling casing (.NET, CSharpier); clean-compile verification (.NET Format task — CSharpier → build → verify — after every edit; tasks.json is canonical); brownfield never licenses silencing diagnostics + suppression scope hierarchy — fix, or [SuppressMessage]+Justification on the symbol (never #pragma), or the owning project's own .editorconfig; root .editorconfig only for genuinely-general rules.
• .vscode/tasks.json: rename .Net* → .NET* (and any launch.json preLaunchTask); add "dependsOrder": "sequence" to the .NET Format task.

3. Skip artifact uploads on PR smoke builds (#179)

• In build-nugetlibrary-task.yml add a smoke input and gate the zip + upload on if: ${{ !inputs.smoke }}; in build-release-task.yml forward smoke: ${{ inputs.smoke }} to the nuget task. (You carry these two; the pypi/executable parts don't apply.) Removes the orphaned release-asset-* uploads that churn the artifact quota on every PR.
• If you're hitting the artifact-storage quota, delete old Actions artifacts (usage recalculates every 6–12h).