projects/forgejo-agent
Watch
5
Fork
You've already forked forgejo-agent
 0
Code Issues 7 Pull requests Packages 2 Activity Actions

feat: make repository/issuenumber inputs in tools optional #141

Merged
davide merged 2 commits from feat/optional-tool-params into main 2026-07-10 23:28:36 +00:00
Conversation 0 Commits 2 Files changed 2 +126 -101
llm-bot commented 2026-07-10 16:49:03 +00:00
Member
Copy link

On long context models get lost and hallucinate names and numbers. If omitted we use the correct one and save some tokens.

Changes

  • Converted all tool definitions (close-issue, create-issue, create-issue-comment, create-pr, create-pr-review) from static exports to factory functions
  • Made repository and issueNumber/prNumber parameters optional in the TypeBox schemas
  • Tools now fall back to the context values (from the current Forgejo event) when these parameters are not provided
  • Updated main.ts to instantiate tools via the new factory functions, passing the repository and issue number from the event context

Benefits

  • Saves tokens in LLM prompts by not requiring the agent to specify repository/issue numbers it already knows
  • Reduces hallucination risk on long-context models that may invent incorrect repository names or issue numbers
On long context models get lost and hallucinate names and numbers. If omitted we use the correct one and save some tokens. ## Changes - Converted all tool definitions (`close-issue`, `create-issue`, `create-issue-comment`, `create-pr`, `create-pr-review`) from static exports to factory functions - Made `repository` and `issueNumber`/`prNumber` parameters optional in the TypeBox schemas - Tools now fall back to the context values (from the current Forgejo event) when these parameters are not provided - Updated `main.ts` to instantiate tools via the new factory functions, passing the repository and issue number from the event context ## Benefits - Saves tokens in LLM prompts by not requiring the agent to specify repository/issue numbers it already knows - Reduces hallucination risk on long-context models that may invent incorrect repository names or issue numbers
feat: make repository/issuenumber inputs in tools optional
All checks were successful
check-commitizen.yaml / Check the PR title (pull_request) Successful in 2s
Details
check-formatter.yaml / Checks with formatters (pull_request) Successful in 6s
Details
check-linter.yaml / Checks with linters (pull_request) Successful in 9s
Details
check-prettier.yaml / Check files formatting (pull_request) Successful in 2s
Details
check-syntax.yaml / Checks with static tools (pull_request) Successful in 16s
Details
check-yamllint.yaml / Lint yaml files (pull_request) Successful in 2s
Details
forgejo-agent / LLM Bot response generation (pull_request) Successful in 2s
Details
cda46214fe
chore: fix some styling
All checks were successful
check-formatter.yaml / Checks with formatters (pull_request) Successful in 6s
Details
check-prettier.yaml / Check files formatting (pull_request) Successful in 2s
Details
check-linter.yaml / Checks with linters (pull_request) Successful in 11s
Details
check-yamllint.yaml / Lint yaml files (pull_request) Successful in 3s
Details
check-syntax.yaml / Checks with static tools (pull_request) Successful in 16s
Details
148ce1b34a
davide scheduled this pull request to auto merge when all checks succeed 2026-07-10 23:28:34 +00:00
davide merged commit 590453c564 into main 2026-07-10 23:28:36 +00:00
davide deleted branch feat/optional-tool-params 2026-07-10 23:28:36 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
projects/forgejo-agent!141
No description provided.