Cleanup AGENTS files

Author: dgarros

AGENTS.md

@@ -64,6 +64,8 @@ cd docs && npm run build              # Build documentation
 - `backend/infrahub/core/protocols.py` – Protocol definitions
 - `frontend/app/src/shared/api/graphql/generated/` – GraphQL types
 - `frontend/app/src/shared/api/rest/types.generated.ts` – REST types
+- `schema/schema.graphql` - GraphQL schema of the Core Schema
+- `schema/openapi.json` - OpenAPI schema for the REST API
 
 Regenerate with: `uv run invoke backend.generate` or `cd frontend/app && npm run codegen`
 
@@ -89,7 +91,7 @@ When editing markdown files (enforced by markdownlint):
 
 - Run formatters before committing (`uv run invoke format`, `npm run biome:fix`)
 - Write tests for new functionality
-- Use type hints (Python) and TypeScript types (frontend)
+- Use type hints for Python (backend) and TypeScript types (frontend)
 
 ### Ask First
 

backend/AGENTS.md

@@ -38,7 +38,7 @@ async def get_node(db: InfrahubDatabase, node_id: str) -> Node:
     await query.execute(db=db)
     return query.get_node()
 
-# ❌ Bad - blocks event loop
+# ❌ Bad - blocks event loop, no type hints
 def get_node(db, node_id):
     return db.get(node_id)
 ```
@@ -50,8 +50,8 @@ def get_node(db, node_id):
 from pydantic import BaseModel, Field
 
 class BranchCreateInput(BaseModel):
-    name: str = Field(..., min_length=1, max_length=250)
-    description: str | None = None
+    name: str = Field(..., min_length=1, max_length=250, description="name of the branch")
+    description: str | None = Field(default=None, description="Description of the branch")
 
 # ❌ Bad
 branch_data = {"name": "feature-x", "description": None}
@@ -102,7 +102,7 @@ class MyQuery(Query):
 
 ## Testing
 
-- Unit tests: no external dependencies, mock everything
+- Unit tests: no external dependencies except database
 - Integration tests: require Neo4j via testcontainers
 - Test files mirror source: `infrahub/core/node.py` → `tests/unit/core/test_node.py`
 - Async tests auto-configured via pytest-asyncio

frontend/app/AGENTS.md

@@ -25,76 +25,3 @@ npm run test:e2e       # E2E tests (Playwright)
 npm run biome:fix      # Format and lint
 npm run codegen        # Regenerate GraphQL types
 ```
-
-## Code Style
-
-### Components
-
-```typescript
-// ✅ Good - Typed props, functional component
-interface NodeCardProps {
-  node: Node;
-  onSelect: (id: string) => void;
-}
-
-export function NodeCard({ node, onSelect }: NodeCardProps) {
-  return (
-    <div className="p-4 rounded-lg" onClick={() => onSelect(node.id)}>
-      {node.display_label}
-    </div>
-  );
-}
-
-// ❌ Bad - Untyped, inline styles
-export function NodeCard(props) {
-  return <div style={{ padding: 16 }}>{props.node.display_label}</div>;
-}
-```
-
-### GraphQL
-
-```typescript
-// ✅ Good - Use generated types
-import type { GetNodesQuery } from "@/shared/api/graphql/generated";
-
-const { data } = useQuery<GetNodesQuery>(GET_NODES);
-```
-
-### Naming Conventions
-
-- **Components:** `PascalCase.tsx`
-- **Hooks:** `useSomething.ts`
-- **Utilities:** `camelCase.ts`
-- **Constants:** `UPPER_SNAKE_CASE`
-
-### Import Order (Biome enforced)
-
-```typescript
-import { useState } from "react";           // React
-import { useQuery } from "@apollo/client";  // External packages
-import { useConfig } from "@/config";       // Internal aliases
-import { NodeCard } from "@/shared/...";    // Shared
-import { useNodeData } from "@/entities/..."; // Entities
-import "./styles.css";                      // Local
-```
-
-## Boundaries
-
-### Always Do
-
-- Run `npm run biome:fix` before committing
-- Use TypeScript types for all props and state
-- Use Tailwind classes (no inline styles)
-- Use generated GraphQL types
-
-### Ask First
-
-- New dependencies
-- New page routes
-- GraphQL query changes affecting multiple components
-
-### Never Do
-
-- Edit files in `src/shared/api/graphql/generated/`
-- Use `console.log` (use `console.error`, `warn`, `info`)
-- Use `any` type without justification