Skip to content

Conversation

@Iamsdt
Copy link
Collaborator

@Iamsdt Iamsdt commented Sep 22, 2025

This pull request makes significant updates to the CI/CD workflows, documentation configuration, and codebase setup. The main goals are to modernize and streamline the continuous integration process, improve documentation structure, update pre-commit hooks, and clean up configuration and code comments for clarity and maintainability.

CI/CD Workflow Modernization:

  • Added a new GitHub Actions workflow .github/workflows/ci.yaml for CI on pull requests to main, including steps for Python dependency caching, environment setup with uv, running pre-commit checks, test execution with coverage, and uploading coverage reports to Codecov.
  • Removed legacy workflows: .github/workflows/formatting_and_pytest.yml and .github/workflows/coverage_report.yml, consolidating formatting, testing, and coverage reporting into the new CI workflow. [1] [2]

Documentation and MkDocs Configuration:

  • Updated mkdocs.yaml to reflect new project branding and repo info, restructured theme and plugin settings, and reorganized markdown extensions for improved documentation navigation and code highlighting. [1] [2] [3]
  • Removed outdated configuration and authentication documentation from Config.md to clean up and clarify supported options.

Pre-commit Hook Updates:

  • Upgraded pre-commit-hooks to v6.0.0, added exclusions for tests/docs/examples, and removed the mypy hook, streamlining pre-commit checks and reducing unnecessary linting overhead.

Codebase and Configuration Cleanups:

  • Changed default API host in pyagenity_api/cli.py from 0.0.0.0 to 127.0.0.1 for improved local security.
  • Integrated ModelResponseConverter for consistent model response handling in agent API calls. [1] [2]
  • Removed unused code and improved docstring formatting in graph/react.py for clarity and maintainability. [1] [2]
  • Improved logging in module loader and worker middleware for better debugging and error tracking. [1] [2] [3]
  • Cleaned up imports and removed unused dependencies from pyagenity_api/src/app/main.py.

Task and Testing Instructions:

  • Added a new Task.md outlining the correct API testing sequence and expectations for error handling during integration tests.

These changes collectively improve the developer experience, reliability of CI/CD, and maintainability of the codebase.

Sarthak422004 pushed a commit to Sarthak422004/pyagenity-api that referenced this pull request Sep 24, 2025
Sarthak422004 and others added 4 commits September 24, 2025 15:15
…vice, and add unit tests for CheckpointerService
…rvice; streamline snowflake ID generator initialization
feat: Refactor CheckpointerService and GraphService, remove ThreadSer…
@Iamsdt Iamsdt merged commit 4e7e546 into main Sep 30, 2025
1 check passed
@Iamsdt Iamsdt deleted the feature/new_version branch September 30, 2025 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants