Skip to content

feat: Add Alquimia AI integration#2734

Open
exengstfeld wants to merge 8 commits into
github:mainfrom
Alquimia-ai:main
Open

feat: Add Alquimia AI integration#2734
exengstfeld wants to merge 8 commits into
github:mainfrom
Alquimia-ai:main

Conversation

@exengstfeld
Copy link
Copy Markdown

Description

Add Alquimia AI integration https://www.alquimia.ai/

Testing

  • [ x ] Tested locally with uv run specify --help
  • [ x ] Ran existing tests with uv sync && uv run pytest
  • [ x ] Tested with a sample project (if applicable)

AI Disclosure

  • [ x ] I did not use AI assistance for this contribution
  • [ ] I did use AI assistance (describe below)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new built-in Alquimia AI integration to Spec Kit, enabling projects to install and use Alquimia’s skills-based command layout alongside existing integrations.

Changes:

  • Introduces AlquimiaAIIntegration (skills layout, context file, skill post-processing for Alquimia-specific frontmatter and argument hints).
  • Registers the new integration and adds it to the built-in integration catalog.
  • Adds a comprehensive integration test suite and updates the speckit workflow compatibility hints to include alquimia.
Show a summary per file
File Description
workflows/speckit/workflow.yml Adds alquimia to the workflow’s advisory integration compatibility list.
tests/integrations/test_integration_alquimia.py New test suite covering registration, setup outputs, context-file behavior (incl. BOM), and skill post-processing behavior.
src/specify_cli/integrations/alquimia_ai/init.py Implements the Alquimia skills integration and skill file post-processing logic.
src/specify_cli/integrations/init.py Imports and registers the new built-in integration.
integrations/catalog.json Adds the new integration entry to the built-in catalog.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 5/5 changed files
  • Comments generated: 4

Comment thread src/specify_cli/integrations/__init__.py
Comment thread src/specify_cli/integrations/alquimia_ai/__init__.py
Comment thread tests/integrations/test_integration_alquimia.py
Comment thread integrations/catalog.json Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot's findings

  • Files reviewed: 5/5 changed files
  • Comments generated: 2

Comment on lines +234 to +235
def test_alquimia_init_remains_usable_when_converter_fails(self, tmp_path):
"""Alquimia init should succeed even without install_ai_skills."""
Comment on lines +105 to +106
def _render_skill(self, template_name: str, frontmatter: dict[str, Any], body: str) -> str:
"""Render a processed command template as a Alquimia skill."""
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