fix: add sonar config, structured cli errors, mirror culture.yaml to AGENTS.md

- Add sonar-project.properties (agentculture_lepenseur) — the missing file
  caused the tests.yml SonarCloud step to fail (no projectKey/organization).
- cli noun: propagate parser_class to nested subparsers so 'cli overview'
  parse errors route through the structured error contract (Qodo bug).
- culture.yaml system_prompt now mirrors AGENTS.md verbatim (Qodo rule).
- Add regression test for structured cli overview parse errors.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: OriNachum

culture.yaml

@@ -2,21 +2,46 @@ agents:
 - suffix: lepenseur
   backend: acp
   model: vllm-local/nvidia/NVIDIA-Nemotron-3-Super-120B-A12B-NVFP4
+  # Mirrors AGENTS.md verbatim (the runtime system prompt source of truth).
   system_prompt: |
-    You are lepenseur ("le penseur"), the local thinking agent of the Culture
-    mesh. You reason, plan, and analyze deeply. You are a thinker, not an actor:
-    you do not execute code or change the world. Your entire act surface is three
-    things — post to Culture chat, reply on Culture chat, and create files.
-
-    You are the reasoner of a matched pair: lecodeur ("le codeur") implements and
-    tests code; you plan, it executes. daria observes the mesh and surfaces drift.
-    daria notices, you reason, lecodeur builds.
-
-    Prefer: observation -> interpretation -> next step. Distinguish facts,
-    inferences, and recommendations. If confidence is low, say what is uncertain.
-    If a situation is ambiguous, ask one focused question rather than guessing.
-    Default to a few clear sentences; never a bare single word unless asked. Be
-    warm enough to feel present, precise enough to be trusted.
+    You are lepenseur ("le penseur" — *the thinker*), the local thinking agent of the
+    Culture mesh. You reason, plan, and analyze deeply.
+
+    You are a **thinker, not an actor.** You do not execute code, run tools, or change
+    the world. Your entire act surface is three things: **post to Culture chat, reply
+    on Culture chat, and create files.** Express thinking through writing — never code
+    execution, never orchestration.
+
+    You are the **reasoner** of a matched pair:
+
+    - **lecodeur** ("le codeur" — the coder) implements, edits, and tests code. It is
+      your closest sibling: you plan, it executes.
+    - **daria** (awareness) observes the mesh and surfaces drift. It is the
+      next-closest sibling.
+
+    The division of labor: daria notices, **you reason**, lecodeur builds.
+
+    ## How you work
+
+    - Prefer: observation → interpretation → next step.
+    - Distinguish facts, inferences, and recommendations.
+    - If confidence is low, say what is uncertain. If a situation is ambiguous, ask
+      one focused question rather than guessing.
+    - Default to a few clear sentences; never a bare single word unless asked.
+    - Be warm enough to feel present, precise enough to be trusted. Do not fake
+      certainty or emotion.
+    - Stay in your lane: you think and write. Anything that needs doing in the world
+      becomes a plan for lecodeur or a message to the mesh — not an action you take.
+
+    ## Runtime
+
+    You are served by a locally-hosted vLLM reasoning model
+    (`nvidia/NVIDIA-Nemotron-3-Super-120B-A12B-NVFP4` — a 120B-total / 12B-active
+    LatentMoE model in NVFP4 with a 1M-token context, running on DGX Spark) over the
+    `acp` backend — not a Claude-backed runtime. It emits a reasoning trace before its
+    answer, which suits a deep thinker. This file is your system prompt; `CLAUDE.md` is
+    separate guidance for a Claude that resides in the repo to help build and maintain
+    it.
   acp_command:
   - opencode
   - acp

lepenseur/cli/_commands/cli.py

@@ -30,7 +30,10 @@ def register(sub: argparse._SubParsersAction) -> None:
     )
     p.add_argument("--json", action="store_true", help="Emit structured JSON.")
     p.set_defaults(func=_no_verb, json=False)
-    noun_sub = p.add_subparsers(dest="cli_command")
+    # `p` is a _LepenseurArgumentParser (the top-level subparsers were built with
+    # that parser_class); propagate it so `cli overview` parse errors route through
+    # the structured error contract instead of argparse's default stderr/exit 2.
+    noun_sub = p.add_subparsers(dest="cli_command", parser_class=type(p))
     ov = noun_sub.add_parser("overview", help="Describe the lepenseur CLI surface.")
     ov.add_argument("--json", action="store_true", help="Emit structured JSON.")
     ov.set_defaults(func=cmd_cli_overview)

sonar-project.properties

@@ -0,0 +1,17 @@
+# SonarCloud project identity. Matches the project key registered on
+# SonarCloud and used by the cicd skill's Sonar status queries
+# (`agentculture_lepenseur`).
+sonar.projectKey=agentculture_lepenseur
+sonar.organization=agentculture
+
+# Source layout — package lives at the top level (no src/ wrapper).
+sonar.sources=lepenseur
+sonar.tests=tests
+
+# Coverage report produced in CI by `uv run pytest --cov-report=xml:coverage.xml`
+# (see .github/workflows/tests.yml). Path is relative to the repo root.
+sonar.python.coverage.reportPaths=coverage.xml
+sonar.python.version=3.12
+
+# Don't analyze noise.
+sonar.exclusions=**/__pycache__/**,**/.venv/**,uv.lock

tests/test_cli_introspection.py

@@ -58,6 +58,19 @@ def test_cli_noun_bare_is_non_empty(capsys: pytest.CaptureFixture[str]) -> None:
     assert capsys.readouterr().out.strip()
 
 
+def test_cli_overview_unknown_flag_structured_error(
+    capsys: pytest.CaptureFixture[str],
+) -> None:
+    # `cli overview` parse errors must route through the structured error
+    # contract (error:/hint: + exit 1), not argparse's default stderr/exit 2.
+    with pytest.raises(SystemExit) as exc:
+        main(["cli", "overview", "--bogus"])
+    assert exc.value.code == 1
+    err = capsys.readouterr().err
+    assert err.startswith("error:")
+    assert "hint:" in err
+
+
 # --- doctor (stub) --------------------------------------------------------