Skip to content

tweak: adjust way skills are presented to agent to increase likelyhood of skill invocations.#17053

Merged
rekram1-node merged 5 commits intodevfrom
improve-skills-logic
Mar 11, 2026
Merged

tweak: adjust way skills are presented to agent to increase likelyhood of skill invocations.#17053
rekram1-node merged 5 commits intodevfrom
improve-skills-logic

Conversation

@rekram1-node
Copy link
Copy Markdown
Collaborator

No description provided.

@rekram1-node rekram1-node changed the title wip tweak: adjust way skills are presented to agent to increase likelyhood of skill invocations. Mar 11, 2026
@rekram1-node
Copy link
Copy Markdown
Collaborator Author

/review

@github-actions
Copy link
Copy Markdown
Contributor

Correction on my earlier comment: I incorrectly flagged an unused import in system.ts. Upon further review, all imports in this PR are correctly used. The pathToFileURL import is in skill/skill.ts (not system.ts) and is used in the new fmt() function.

Overall, this PR follows the style guide well:

  • Good use of short variable names (list)
  • Proper early returns
  • No unnecessary destructuring
  • Good refactoring to avoid duplication via Skill.available() and Skill.fmt()

lgtm

@rekram1-node rekram1-node merged commit 0f6bc8a into dev Mar 11, 2026
8 checks passed
@rekram1-node rekram1-node deleted the improve-skills-logic branch March 11, 2026 15:24
demostanis pushed a commit to demostanis/opencode that referenced this pull request Mar 19, 2026
demostanis pushed a commit to demostanis/opencode that referenced this pull request Mar 20, 2026
balcsida pushed a commit to balcsida/opencode that referenced this pull request Mar 24, 2026
balcsida pushed a commit to balcsida/opencode that referenced this pull request Apr 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant