mirror of
https://github.com/microsoft/BCQuality.git
synced 2026-08-06 17:36:53 +01:00
Foreslaa regel: Agent must resolve developer identity from BC
This commit is contained in:
parent
aad03770fa
commit
cec927df81
1 changed files with 48 additions and 0 deletions
|
|
@ -0,0 +1,48 @@
|
|||
---
|
||||
rule-id: CURABIS-MCP-007
|
||||
title: Agent must resolve developer identity from BC
|
||||
category: mcp
|
||||
severity: warning
|
||||
applies-to: [agent-files, bc-mcp]
|
||||
bc-version: [all]
|
||||
---
|
||||
|
||||
# Agent must resolve developer identity from BC
|
||||
|
||||
## Rule
|
||||
|
||||
Agent files must not contain static employee-to-code mappings.
|
||||
Developer identity must always be resolved at runtime from the BC users tool (PAG6102903).
|
||||
|
||||
## Rationale
|
||||
|
||||
Employee data is owned by the company, not by any individual project. A static mapping
|
||||
in a project-level agent file duplicates company data and will silently drift on every
|
||||
personnel change -- a new hire is missing, a former employee remains listed.
|
||||
|
||||
The BC users page (PAG6102903) is the single source of truth for employeeCode + name
|
||||
+ userId mapping. Resolving identity at runtime ensures attribution is always correct
|
||||
without any maintenance overhead on the project side.
|
||||
|
||||
## What this prevents
|
||||
|
||||
- Incorrect task attribution after an employee leaves or changes role
|
||||
- N agent files requiring manual update for a single personnel change
|
||||
- Silent drift where an agent signs comments with the wrong name
|
||||
|
||||
## Correct pattern
|
||||
|
||||
In bc-mcp agent: always resolve at runtime.
|
||||
git config user.email -> look up via users tool (PAG6102903) -> employeeCode + name
|
||||
|
||||
## Incorrect pattern
|
||||
|
||||
Static employee tables in agent files are forbidden:
|
||||
|
||||
| MID | Michael Dieringer | Developer |
|
||||
| LIT | Linh | Consultant |
|
||||
|
||||
## Exceptions
|
||||
|
||||
None. If the users tool is temporarily unavailable, say so and stop -- do not fall back
|
||||
to a hardcoded table.
|
||||
Loading…
Add table
Add a link
Reference in a new issue