Skip to content

Fix skills documentation: correct paths, caching behavior, and tab formatting#399

Merged
bpulluta merged 2 commits intofeature/skills-prfrom
copilot/sub-pr-397
Mar 17, 2026
Merged

Fix skills documentation: correct paths, caching behavior, and tab formatting#399
bpulluta merged 2 commits intofeature/skills-prfrom
copilot/sub-pr-397

Conversation

Copy link
Contributor

Copilot AI commented Mar 17, 2026

Nine inaccuracies in the one-shot skills docs identified by automated review: broken file paths, a mischaracterized caching flag, incorrect pipeline sequencing description, and literal tab characters causing inconsistent Markdown rendering.

yaml-setup/SKILL.md

  • Canonical reference path updated from non-existent examples/one_shot_schema_extraction/<tech>_plugin_config.yaml to actual examples/one_shot_schema_extraction/plugin_config.yaml; clarifies <tech>_plugin_config.yaml is the naming convention for new tech-specific configs, not the existing examples
  • cache_llm_generated_content description corrected to include heuristic_keywords — all three generated artifacts (query_templates, website_keywords, heuristic_keywords) are cached per compass/plugin/one_shot/base.py
  • Example reference fixed from non-existent compass/extraction/geothermal_electricity/geothermal_plugin_config.yamlcompass/extraction/ghp/plugin_config.yaml

web-scraper/SKILL.md

  • Canonical reference updated to point to actual files in examples/
  • Website crawl phase description corrected: runs only if the SE phase did not yield an ordinance context (not unconditionally after SE)
  • Literal tabs replaced with spaces in tuning loop section

extraction-run/SKILL.md

  • Develop path corrected from non-existent examples/one_shot_schema_extraction_<tech>/examples/one_shot_schema_extraction/
  • Promotion guidance updated to include the required Python registration step: create __init__.py using create_schema_based_one_shot_extraction_plugin and add an import in compass/extraction/__init__.py (mirrors compass/extraction/ghp/__init__.py)
  • Literal tabs replaced with spaces throughout

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Co-authored-by: bpulluta <115118857+bpulluta@users.noreply.github.com>
Copilot AI requested review from castelao and ppinchuk as code owners March 17, 2026 22:24
Copilot AI changed the title [WIP] Add one-shot skills reliability and guardrails Fix skills documentation: correct paths, caching behavior, and tab formatting Mar 17, 2026
Copilot AI requested a review from bpulluta March 17, 2026 22:25
@bpulluta bpulluta merged commit 81fcbff into feature/skills-pr Mar 17, 2026
@bpulluta bpulluta deleted the copilot/sub-pr-397 branch March 17, 2026 22:26
ppinchuk pushed a commit that referenced this pull request Mar 26, 2026
* Add COMPASS workflow skills

* Added one-shot skills

* update one-shot SKILL.md structure and trigger contracts

* Initial plan (#398)

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

* Fix skills documentation: correct paths, caching behavior, and tab formatting (#399)

* Initial plan

* Fix all review comments in skills documentation

Co-authored-by: bpulluta <115118857+bpulluta@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: bpulluta <115118857+bpulluta@users.noreply.github.com>

* renamed skills and fixed minor comments

* udpated skills Paul review march 26

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
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.

2 participants