Skip to content

Refactor tool preflight protocol#78

Merged
F16shen merged 1 commit intoAI-Shell-Team:mainfrom
sonald:main
Mar 23, 2026
Merged

Refactor tool preflight protocol#78
F16shen merged 1 commit intoAI-Shell-Team:mainfrom
sonald:main

Conversation

@sonald
Copy link
Copy Markdown
Collaborator

@sonald sonald commented Mar 22, 2026

Summary

  • Problem: The execution logic for tools and the LLM session workflow are currently coupled together, which violates design specifications and hinders scalability. All of these components should be fully abstracted from the LLM session.
  • Changes:
  • Related Issue: #

Change Type

  • Bug fix
  • Feature
  • Refactor
  • Docs
  • Other

Scope

  • Core shell / PTY
  • AI agent / LLM
  • Skills / Tools
  • Security
  • Configuration
  • CLI / Interface
  • Packaging / Installation
  • CI/CD
  • Documentation

User-visible Changes

Compatibility

  • Backward compatible? (Yes/No)
  • Config changes? (Yes/No - if yes, describe migration)

Testing

Checklist

  • Code follows project style
  • Tests added if needed
  • Documentation updated if needed

@github-actions
Copy link
Copy Markdown
Contributor

Thanks for the pull request. A maintainer will review it when available.

Please keep the PR focused, explain the why in the description, and make sure local checks pass before requesting review.

Contribution guide: https://github.com/AI-Shell-Team/aish/blob/main/CONTRIBUTING.md

@github-actions
Copy link
Copy Markdown
Contributor

This pull request description looks incomplete. Please update the missing sections below before review.

Missing items:

  • User-visible Changes
  • Testing

@github-actions github-actions Bot added pty PTY or interactive process issue core agent Agent or LLM workflow issue tools tests size: L labels Mar 22, 2026
@F16shen F16shen merged commit 54d80d2 into AI-Shell-Team:main Mar 23, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent Agent or LLM workflow issue core pty PTY or interactive process issue size: L tests tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants