Skip to content

feat(skills): add auto-optimize workflow - #1428

Merged
Qiong Wu (qiowu) (DingmaomaoBJTU) merged 10 commits into
microsoft:mainfrom
DingmaomaoBJTU:dingmaomaobjtu/add-auto-optimize-skill
Sep 18, 2026
Merged

Qiong Wu (qiowu) (DingmaomaoBJTU) merged 10 commits into
microsoft:mainfrom
DingmaomaoBJTU:dingmaomaobjtu/add-auto-optimize-skill

Conversation

@DingmaomaoBJTU

Copy link
Copy Markdown
Collaborator

Summary

  • check in the canonical auto-optimize Agent Skill from gim-home/ModelKitArtifacts@2bcf29d9150ab3af2909f3cc2a0b278d7ca2387d
  • adapt the repository-layout contract and register skills/auto-optimize/tests in the existing CI matrix, following feat(skills): add model support contribution workflow #1415
  • normalize imported Python to this repository's Ruff policy while preserving script interfaces and generated output
  • close inherited fail-open validation paths for non-finite hotspot evidence and quoted/escaped absolute reproduction paths

Validation

  • uv run pytest skills/auto-optimize/tests --tb=short --no-cov (309 passed)
  • uv run ruff check skills/auto-optimize
  • uv run ruff format --check skills/auto-optimize
  • pre-commit hooks on all 13 target-adapted files
  • all 7 bundled JSON documents parsed successfully
  • all 29 pinned-source files are present; final diff is limited to the skill and CI registration

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Detailed readiness review: all 309 bundled tests passed locally, and all reported CI checks are green. However, the executable validation and workflow ordering gaps below prevent a ready-for-check-in verdict. Additional local reproductions confirmed acceptance of NaN final-report evidence and a generated replay wrapper returning success after an earlier native failure. No PR files were modified. The current tests provide useful script/contract coverage, but do not cover these end-to-end failure paths.

Comment thread skills/auto-optimize/scripts/render_report.py
Comment thread skills/auto-optimize/scripts/finalize_output.py
Comment thread skills/auto-optimize/SKILL.md Outdated
@DingmaomaoBJTU
Qiong Wu (qiowu) (DingmaomaoBJTU) marked this pull request as ready for review September 18, 2026 02:39
@DingmaomaoBJTU
Qiong Wu (qiowu) (DingmaomaoBJTU) merged commit 4b83a5f into microsoft:main Sep 18, 2026
9 checks passed
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