Skip to content

feat: add agentify list - #66

Merged
sbillinge merged 5 commits into
diffpy:mainfrom
ycexiao:agentify-list
Nov 14, 2026
Merged

sbillinge merged 5 commits into
diffpy:mainfrom
ycexiao:agentify-list

Conversation

@ycexiao

@ycexiao ycexiao commented Nov 13, 2026

Copy link
Copy Markdown
Contributor
  • feat: add agentify list
  • feat: support --skill for both agentify and agentify list

What problem does this PR address?

Closes #63

deploy subcommand is not implemented because the agentify command itself works as the deploy command.

What should the reviewer(s) do?

ycexiao and others added 2 commits November 13, 2026 10:47
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.69%. Comparing base (87d82aa) to head (2ed464d).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   99.69%   99.69%           
=======================================
  Files           8        8           
  Lines         648      666   +18     
=======================================
+ Hits          646      664   +18     
  Misses          2        2           
Files with missing lines Coverage Δ
tests/conftest.py 100.00% <100.00%> (ø)
tests/test_agentify.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ycexiao
ycexiao marked this pull request as ready for review November 13, 2026 16:04
@ycexiao

ycexiao commented Nov 13, 2026

Copy link
Copy Markdown
Contributor Author

@sbillinge it's ready for review.

Comment thread src/diffpy/apps/app_agentify.py Resolved


def list_skills(args):
console = Console(

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Use the same prompt package that cookiecuter and scikit-package use.

@sbillinge

Copy link
Copy Markdown
Contributor

@ycexiao this looks good but is failing tests

@ycexiao

ycexiao commented Nov 14, 2026

Copy link
Copy Markdown
Contributor Author

@sbillinge The test is fixed. It is ready for review.

@sbillinge

Copy link
Copy Markdown
Contributor

@ycexiao I like these changes. Seems like a good way to handle this, though I am not qualified to comment as I don't really know how agents will interact with this.

@sbillinge
sbillinge merged commit f1d2f61 into diffpy:main Nov 14, 2026
7 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.

feat: diffpy-app agentify list and diffpy-app agentify deploy <skill-name> to list and to deploy skills

2 participants