Skip to content

Conversation

@armando-salazar
Copy link

@armando-salazar armando-salazar commented May 16, 2025

This PR adds comprehensive guides for the API documentation system to the manual. The guides explain how to document APIs in the iR Engine codebase.

Changes

  • Added a new section under modules called "API Documentation System"

  • Created two main guides:

    • Documentation System: Explains the architecture and components of the API documentation system
    • Documentation Guide: Provides step-by-step instructions for documenting API endpoints
  • Updated navigation and cross-references throughout the documentation

  • Organized content to clearly reflect its focus on documenting APIs

Testing

  • Verified all links work correctly
  • Ensured proper formatting and structure
  • Confirmed navigation is updated in summary.md

@armando-salazar armando-salazar self-assigned this May 16, 2025
@armando-salazar armando-salazar added the documentation Improvements or additions to documentation label May 16, 2025
- Example: `packages/common/src/schemas/cluster/build-status.schema.ts`
- Contains path definitions, method lists, and data schemas

2. **Documentation files** (`packages/server-core/src/[category]/[name]/[name].docs.ts`):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we stress the importance of this step more? Although feathers/swagger automates most of the documentation, without developer descriptions, examples and parameter descriptions the documentation generated is not the most useful since descriptions default to a very generic message such as Creates a new resource with data.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i.e.
Screenshot 2025-05-16 at 1 48 25 PM
Screenshot 2025-05-16 at 1 48 01 PM

@armando-salazar armando-salazar requested a review from sfgarza May 21, 2025 00:45
@armando-salazar armando-salazar merged commit 0de4bdd into main May 21, 2025
@armando-salazar
Copy link
Author

Enhanced API documentation guides with workflow details and improved emphasis on proper descriptions. Added information about hooks, validators, and resolvers based on developer feedback. Also included details about API organization and the standard development workflow.

@armando-salazar armando-salazar deleted the new-api-swagger-workflow-docs branch May 21, 2025 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants