Skip to content

feat: Add Alquimia AI integration#2734

Open
exengstfeld wants to merge 13 commits into
github:mainfrom
Alquimia-ai:alquimia-integration-merge-conflicts-resolution
Open

feat: Add Alquimia AI integration#2734
exengstfeld wants to merge 13 commits into
github:mainfrom
Alquimia-ai:alquimia-integration-merge-conflicts-resolution

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)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 Outdated
Comment thread tests/integrations/test_integration_alquimia.py
Comment thread integrations/catalog.json Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 thread tests/integrations/test_integration_alquimia.py Outdated
Comment thread src/specify_cli/integrations/alquimia_ai/__init__.py Outdated

@mnriem mnriem left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please address Copilot feedback. If not applicable, please explain why

@exengstfeld

Copy link
Copy Markdown
Author

Please address Copilot feedback. If not applicable, please explain why

PR ready for review :)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 thread src/specify_cli/integrations/alquimia_ai/__init__.py Outdated
Comment thread src/specify_cli/integrations/alquimia_ai/__init__.py
@mnriem

mnriem commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Please address Copilot feedback

exengstfeld and others added 2 commits June 10, 2026 16:24
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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: 3

Comment thread src/specify_cli/integrations/alquimia_ai/__init__.py Outdated
Comment thread integrations/catalog.json
Comment thread workflows/speckit/workflow.yml
@mnriem

mnriem commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Please address Copilot feedback

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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: 4

Comment on lines +5 to +7
import re
from pathlib import Path
from typing import Any
Comment on lines +14 to +20
# Note injected into hook sections so Alquimia maps dot-notation command
# names (from extensions.yml) to the hyphenated skill names it uses.
_HOOK_COMMAND_NOTE = (
"- When constructing slash commands from hook command names, "
"replace dots (`.`) with hyphens (`-`). "
"For example, `speckit.git.commit` → `/speckit-git-commit`.\n"
)
Comment on lines +171 to +176
def post_process_skill_content(self, content: str) -> str:
"""Inject Alquimia-specific frontmatter flags and hook notes."""
updated = self._inject_frontmatter_flag(content, "user-invocable")
updated = self._inject_frontmatter_flag(updated, "disable-model-invocation", "false")
updated = self._inject_hook_command_note(updated)
return updated
Comment thread integrations/catalog.json
@@ -3,6 +3,15 @@
"updated_at": "2026-06-02T00:00:00Z",
@mnriem

mnriem commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Please address Copilot feedback and resolve conflicts and pull in upstream/main

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