Add dotnet format pre-commit hook and CLAUDE.md symlink (#753)

* Add dotnet format pre-commit hook and CLAUDE.md symlink

- Symlink CLAUDE.md -> AGENTS.md so Claude Code loads the existing
  agent instructions (mirrors the .aider.md/.clinerules/.cursorrules pattern)
- Replace the buried "Run dotnet format before commit" line in AGENTS.md
  with a dedicated Formatting section that spells out the exact command
  (whitespace + style only; analyzers is slow/noisy) and flags merge
  commits as a known style-drift source
- Add a local pre-commit hook that runs both subcommands with
  --verify-no-changes when .cs files are staged, so drift fails the
  commit instead of being noticed in CI or review

Run 'pre-commit install' once per clone to activate the hook.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* chore: cleanup after precommit hook updates

Signed-off-by: Andrew Berezovskyi <andriib@kth.se>

---------

Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

Author: berezovskyi

.coderabbit.yaml

@@ -19,7 +19,7 @@ reviews:
   path_instructions:
     - path: "**/*.cs"
       instructions: |
-        Review the C# code against the Microsoft Framework design guidelines as well as 10 guidelines 
+        Review the C# code against the Microsoft Framework design guidelines as well as 10 guidelines
         from "Building Maintainable Software" by Joost Visser. Point out deviations and improvements.
     - path: "**/*.md"
       instructions: |

.pre-commit-config.yaml

@@ -1,10 +1,18 @@
 repos:
 - repo: https://github.com/gitleaks/gitleaks
-  rev: v8.16.3
+  rev: v8.30.0
   hooks:
   - id: gitleaks
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v4.4.0
+  rev: v6.0.0
   hooks:
   - id: end-of-file-fixer
   - id: trailing-whitespace
+- repo: local
+  hooks:
+  - id: dotnet-format
+    name: dotnet format (whitespace + style, verify only)
+    entry: bash -c 'cd OSLC4Net_SDK && dotnet format whitespace . --verify-no-changes && dotnet format style . --verify-no-changes --no-restore'
+    language: system
+    pass_filenames: false
+    files: '\.cs$'

.vscode/settings.json

@@ -1,3 +1,3 @@
 {
    "dotnet.defaultSolution": "OSLC4Net_SDK/OSLC4Net.Core.slnx"
-}
+}

.vscode/tasks.json

@@ -38,4 +38,4 @@
          "problemMatcher": "$msCompile"
       }
    ]
-}
+}

AGENTS.md

@@ -56,8 +56,6 @@ Do not inline the `AGENT_BUILD` variable!
 
 You must be CD'd into `OSLC4Net_SDK` or subfolder before running `dotnet` commands.
 
-Run `dotnet format` before commit.
-
 ### Test Filters
 
 Running TUnit via `dotnet run` supports test filters.
@@ -99,6 +97,19 @@ TUnit also supports filtering by your own [properties](https://thomhurst.github.
 
 And if your test had a property with the name "MyFilterName" and its value contained "SomeValue", then your test would be executed.
 
+## Formatting (required before every commit)
+
+Run from the repo root:
+
+```bash
+dotnet format whitespace ./OSLC4Net_SDK && dotnet format style ./OSLC4Net_SDK --no-restore
+```
+
+- Use `whitespace` + `style` only. Skip `analyzers` — it runs third-party rules (Meziantou etc.) and is slow/noisy.
+- `--no-restore` on the second call avoids a redundant restore.
+- Also run this **after resolving merge conflicts**. Hand-edited hunks re-introduce style drift even when both parents were clean.
+- A pre-commit hook (`.pre-commit-config.yaml`) enforces this; install once with `pre-commit install`.
+
 ## Project Structure
 
 - `OSLC4Net_SDK/` - Main SDK projects (.NET 10)

CLAUDE.md

@@ -0,0 +1 @@
+AGENTS.md

OSLC4Net_SDK/.config/dotnet-tools.json

@@ -10,4 +10,4 @@
       "rollForward": false
     }
   }
-}
+}

OSLC4Net_SDK/.dockerignore

@@ -27,4 +27,4 @@ README.md
 !.git/HEAD
 !.git/config
 !.git/packed-refs
-!.git/refs/heads/**
+!.git/refs/heads/**

OSLC4Net_SDK/Examples/OSLC4Net.Client.Jazz/Oslc/Jazz/JazzConstants.cs

@@ -4,7 +4,7 @@
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * and Eclipse Distribution License v. 1.0 which accompanies this distribution.
- *  
+ *
  * The Eclipse Public License is available at http://www.eclipse.org/legal/epl-v10.html
  * and the Eclipse Distribution License is available at
  * http://www.eclipse.org/org/documents/edl-v10.php.

OSLC4Net_SDK/Examples/OSLC4Net.Client.Jazz/Oslc/Jazz/JazzFormAuthClient.cs

@@ -4,7 +4,7 @@
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * and Eclipse Distribution License v. 1.0 which accompanies this distribution.
- *  
+ *
  * The Eclipse Public License is available at http://www.eclipse.org/legal/epl-v10.html
  * and the Eclipse Distribution License is available at
  * http://www.eclipse.org/org/documents/edl-v10.php.
@@ -59,7 +59,7 @@ public JazzFormAuthClient(String url, String user, String password, ILogger<Oslc
     /// Create a new Jazz Form Auth client for the given URL, user and password
     /// </summary>
     /// <param name="url">the URL of the Jazz server, including the web app context</param>
-    /// <param name="authUrl">the base URL to use for authentication.  This is normally the 
+    /// <param name="authUrl">the base URL to use for authentication.  This is normally the
     /// application base URL for RQM and RTC and is the JTS application URL for fronting
     /// applications like RRC and DM.</param>
     /// <param name="user"></param>