Skip to content

Fix Alquimia argument-hint injection for folded descriptions - #4063

Merged
mnriem merged 1 commit into
github:mainfrom
WOLIKIMCHENG:fix/alquimia-folded-argument-hint
Aug 12, 2026
Merged

mnriem merged 1 commit into
github:mainfrom
WOLIKIMCHENG:fix/alquimia-folded-argument-hint

Conversation

@WOLIKIMCHENG

Copy link
Copy Markdown
Contributor

Summary

  • Fix Alquimia skill frontmatter injection so argument-hint is inserted after the full YAML description scalar.
  • Add folded, quoted folded, and multi-paragraph regression coverage matching Claude’s existing behavior.

Tests

  • .venv/bin/python -m pytest tests/integrations/test_integration_alquimia.py tests/integrations/test_integration_claude.py tests/integrations/test_integration_base_skills.py -q
  • uvx ruff@0.15.0 check src tests
  • git diff --check
  • .venv/bin/python -m pytest tests/integrations -q

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

Fixes Alquimia’s argument-hint injection for multiline YAML descriptions.

Changes:

  • Skips folded and blank scalar continuation lines before injection.
  • Adds regression tests for plain, quoted, and multi-paragraph descriptions.
Show a summary per file
File Description
src/specify_cli/integrations/alquimia/__init__.py Makes hint injection fold-aware.
tests/integrations/test_integration_alquimia.py Covers multiline description variants.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@mnriem
mnriem merged commit b77ca57 into github:main Aug 12, 2026
14 checks passed
@mnriem

mnriem commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Thank you!

kanfil added a commit to tikalk/agentic-sdlc-spec-kit that referenced this pull request Aug 12, 2026
… with main

- fix: Alquimia argument hints after folded descriptions (github#4063)
- Merge origin/main to incorporate PR #109/#110/#111 merge commits
  (0.16.0+adlc3 release tagged on main)

Assisted-by: opencode (model: glm-5.2, supervised)
ira-at-work added a commit to ira-at-work/spec-kit that referenced this pull request Aug 17, 2026
…065-stable-block-identifiers

* 'main' of https://github.com/github/spec-kit:
  fix(bundler): treat a blank active integration as indeterminate in FR-019 (github#3886)
  Integrate Junie with dot-to-hyphen behavior and command formatting  (github#4073)
  Update A11Y Governance preset to v0.4.3 (github#4074)
  Fix Alquimia argument hints after folded descriptions (github#4063)
  fix: use bounded read for bundle download HTTP responses (github#3764)
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