This commit refines the project documentation by adding result verification steps to all recipes in the cookbook and formalizing documentation standards in the project guidelines.
Changes:
- docs/howtos/cookbook.md:
- Added 'Verify' steps to all recipes using GraphQL queries or agent checks.
- Enhanced 'Extending Schema with a New Device Type' recipe:
- Updated Step 1 with a realistic schema using 'DistanceTrackerDevice' implementing 'Host' and 'Device' interfaces.
- Added mandatory advice for new device types to implement 'Host' and 'Device'.
- Referenced the existing 'location_tracker_devices.graphql' as a prepared real-world sample.
- Split Step 3 into Method A (Manual Creation) and Method B (Automated Import).
- Expanded Step 4 with a host creation step (using importHosts) and a filtered allDevices query using tag_deviceType.
- Standardized icons for preparation (🛠️), configuration (⚙️), execution (🚀), and verification (✅).
- .junie/guidelines.md:
- Added 'Documentation Templates' section defining a formal Recipe Template.
- Standardized icons and step naming to maintain visual consistency across all guides.
- Added 'Review & Approval' rule to prevent automatic commits without user review.
- readme.improvement.plan.md:
- Added and completed 'Priority 7: Standardization & Verification' tasks.
- Updated implementation order to reflect the new standardization phase.
5.3 KiB
5.3 KiB
Project Guidelines
This document provides concise information and best practices for developers working on the Zabbix GraphQL API project.
Roadmap
The Roadmap is to be considered as outlook giving constraints on architectural and design decisions for current work on the project.
Environment
- Operating System: Windows with WSL + Ubuntu installed.
- Commands: Always execute Linux commands (e.g. use
lsinstead ofdir).
Tech Stack
- Runtime: Node.js (v24+)
- Language: TypeScript (ESM)
- API: GraphQL (Apollo Server 4)
- Testing: Jest
- Deployment: Docker
Project Structure
src/api/: GraphQL server configuration, schema loading, and root resolvers (seecreateResolversinresolvers.ts).src/datasources/: Modular classes for interacting with various Zabbix API components (hosts, items, etc.).src/execution/: Business logic for complex, multi-step operations (importers, exporters, deleters).src/model/: Shared data models and enumerations.src/test/: Unit and integration tests.schema/: GraphQL Schema Definition Language (SDL) files.
Common Scripts
npm run start: Launches the development server withtsxandnodemonfor hot-reloading.npm run test: Executes the Jest test suite.npm run codegen: Generates TypeScript types based on the GraphQL schema definitions.npm run compile: Compiles TypeScript source files into thedist/directory.npm run prod: Prepares the schema and runs the compiled production build.
Best Practices & Standards
- ESM & Imports: The project uses ECMAScript Modules (ESM). Always use the
.jsextension when importing local files (e.g.import { Config } from "../common_utils.js";), even though the source files are.ts. - Configuration: Always use the
Configclass to access environment variables. Avoid directprocess.envcalls. - Type Safety: Leverage types generated via
npm run codegenfor resolvers and data handling to ensure consistency with the schema. - Import Optimization:
- Always optimize imports before committing.
- Project setting
OPTIMIZE_IMPORTS_BEFORE_PROJECT_COMMITis enabled.
- Modular Datasources: When adding support for new Zabbix features, create a new datasource class in
src/datasources/extendingZabbixRESTDataSource. - Schema Organization: Place GraphQL SDL files in the
schema/directory. Use descriptive comments in SDL as they are used for API documentation. - Testing: Write reproduction tests for bugs and cover new features with both unit and integration tests in
src/test/. - Grammar & Style: Avoid using a comma after "e.g." or "i.e." (e.g. use "e.g. example" instead of "e.g., example").
Documentation Style
- Bullet Points: Use bullet points instead of enumerations for lists to maintain consistency across all documentation.
- Visual Style: Use icons in headers and bold subjects for primary list items (e.g.
- **Feature**: Description) to match theREADME.mdstyle.- Note: Standardize colon placement (outside bold tags) for primary list items.
- Sub-points: Use indented bullet points for detailed descriptions or additional context.
- Format: Use the
- *Subject*: Descriptionformat for specific references or categorized sub-items (e.g.- *Reference*: ...). - Format: Use plain text for general descriptions that follow a primary list item.
- Format: Use the
- Formatting: Avoid extra blank lines between headers and list items.
Documentation Templates
Cookbook & Recipes
Follow this template for all cookbook entries to ensure consistency and AI-parsability.
Recipe Template
## 🍳 Recipe: [Action-Oriented Title]
[Brief description of what this recipe achieves.]
### 📋 Prerequisites
- [Constraint 1]
- [Constraint 2]
### 🛠️ Step 1: [Preparation/Definition]
[Instructions for preparing data or environment.]
### ⚙️ Step 2: [Configuration/Settings]
[Instructions for configuring settings or environment variables.]
### 🚀 Step 3: [Execution/Action]
[The main GraphQL operation or command to execute.]
### ✅ Step 4: [Verification]
[How to verify that the operation was successful using a query or agent check.]
Standard Icons for Steps
- 🍳 Recipe: Main header icon.
- 📋 Prerequisites: Necessary conditions before starting.
- 🛠️ Preparation: Initial setup, data definition, or file creation.
- ⚙️ Configuration: Changes to settings,
.env, or Zabbix UI. - 🚀 Execution: The primary action or mutation.
- ✅ Verification: Result checking and outcome validation.
- 💡 Tip/Alternative: Helpful hints or alternative methods.
- 🤖 AI/MCP: Agent-specific instructions.
Git Standards
- Commit Messages: Use Conventional Commits (e.g.
feat:,fix:,chore:,docs:,test:,refactor:,style:).- If a commit is complex and covers different aspects, the message must always contain a detailed list of what was changed within the optional "body" section, in addition to the short "description" in the header.
- Review & Approval: Never commit changes automatically. Always present the proposed changes and the updated plan to the user for review. Allow the user to provide feedback or add tasks to the plan before proceeding with a commit.