-
Notifications
You must be signed in to change notification settings - Fork 1
Patch/sync upstream 2026 01 15 #55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…(#2127) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
…for shell commands (#2145) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
…istic cwd (#2150)
Co-authored-by: Amit Singh <amitksingh1490@gmail.com> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
…ng (#2152) Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
…and module consolidation (#2203)
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
…(#2222) Co-authored-by: ForgeCode <noreply@forgecode.dev> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: ForgeCode <noreply@forgecode.dev>
Co-authored-by: ForgeCode <noreply@forgecode.dev> Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com> Co-authored-by: Tushar Mathur <tusharmath@gmail.com>
Co-authored-by: ForgeCode <noreply@forgecode.dev>
|
Todo: migrate histfile commits, and cleanup on changes. |
| match self { | ||
| ToolOperation::FsRead { input: _, output: _ } => None, | ||
| ToolOperation::ImageRead { output: _ } => None, | ||
| ToolOperation::FsCreate { input, output } => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We do want previews for fs create
crates/paws_app/src/tool_executor.rs
Outdated
| } | ||
| ToolCatalog::SemSearch(_) => { | ||
| // Semantic search is not implemented in Paws yet | ||
| anyhow::bail!("Semantic search is not yet supported") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just drop the tool entirely
| ("search", ToolName::new("fs_search")), | ||
| ("Read", ToolName::new("read")), | ||
| ("Write", ToolName::new("write")), | ||
| ]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do a clean replace, no part should refer to dpericated names
| .map(|pattern| SummaryTool::Search { pattern }), | ||
| ToolCatalog::SemSearch(input) => Some(SummaryTool::Search { | ||
| pattern: input | ||
| .queries |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also drop
|
Keep print conversation |
|
Bug: Spinner stuck on loading after dump conversation |
- Added `SetMinimaxParams` transformer to handle specific Minimax model parameters (temperature, top_p, top_k). - Integrated Minimax transformer into the OpenAI provider pipeline. - Added `input_modalities` field to `paws_domain::Model` to track supported input types (Text, Image). - Updated Anthropic, Gemini, and OpenAI model DTOs to populate `input_modalities`. - Cleaned up unused snapshots and benchmark files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR syncs 114 upstream commits from the Forge repository to the Paws fork (as of 2026-01-15), applying bug fixes, new features, and improvements while maintaining fork-specific customizations. The changes are split into batches to manage risk.
Changes:
- Removes semantic search tool (
sem_search) and templates - Adds image/PDF reading support to
fs_readtool with MIME detection - Adds validation framework with
SyntaxErrortype for file operations - Introduces permission checking system and restricted mode support
- Updates tool descriptions to use external markdown files with template rendering
- Adds HTTP 522 to retry status codes and helper functions
Reviewed changes
Copilot reviewed 180 out of 181 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/migrations/*.md | Documents the sync process, batches, and migration history |
| templates/paws-partial-summary-frame.md | Removes sem_search template block |
| crates/paws_domain/src/validation.rs | Adds SyntaxError type for validation |
| crates/paws_services/src/tool_services/fs_read.rs | Adds image/PDF support, line truncation, MIME detection |
| crates/paws_services/src/policy.rs | Removes message field from Execute operation |
| crates/paws_domain/src/tools/descriptions/*.md | New external tool description files |
| crates/paws_app/src/template_engine.rs | Adds 'contains' handlebars helper |
| crates/paws_app/src/tool_resolver.rs | Adds tool aliases for backward compatibility |
| crates/paws_domain/src/model.rs | Adds input_modalities field to Model |
| crates/paws_common/src/json_repair/*.rs | Adds schema coercion and fixes regex parser |
| .github/workflows/release.yml | Removes homebrew release job |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "PDF exceeds size limit. Use a smaller PDF or increase FORGE_MAX_IMAGE_SIZE." | ||
| } else { | ||
| "Image exceeds size limit. Compress the image or increase FORGE_MAX_IMAGE_SIZE." | ||
| } |
Copilot
AI
Jan 26, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The error message contains "FORGE_MAX_IMAGE_SIZE" which should be changed to "PAWS_MAX_IMAGE_SIZE" to match the Paws fork naming convention.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot open a new pull request to apply changes based on this feedback
| "cwd": { | ||
| "description": "The working directory where the command should be executed.", | ||
| "type": "string" | ||
| "description": "The working directory where the command should be executed. If not specified, defaults to the current working directory from the environment.", | ||
| "type": "string", | ||
| "nullable": true |
Copilot
AI
Jan 26, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Shell tool's cwd parameter is now optional (nullable) but the description says "If not specified, defaults to the current working directory from the environment." This is correct, but ensure all call sites handle the Option correctly and default to the environment cwd when None.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot open a new pull request to apply changes based on this feedback, please do check skip opening the PR if is found consistent
| - fetch | ||
| - read_image | ||
| - search |
Copilot
AI
Jan 26, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The read_image tool has been removed from the agents (sage.md line 10, muse.md line 13) but is still present in paws.md line 18 and the implementation still exists in the codebase. This is inconsistent. Either remove read_image from all agents including paws.md, or add it back to sage and muse if image reading is still supported.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot open a new pull request to apply changes based on this feedback, keeping read_image
echo
Outdated
| a | ||
| b | ||
| c | ||
| d |
Copilot
AI
Jan 26, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file named "echo" appears to be accidentally committed. It contains only test data (a, b, c, d) and doesn't have a meaningful extension or purpose in the repository structure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot open a new pull request to apply changes based on this feedback
|
@manthanabc I've opened a new pull request, #74, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@manthanabc I've opened a new pull request, #75, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@manthanabc I've opened a new pull request, #76, to work on those changes. Once the pull request is ready, I'll request review from you. |
|
@manthanabc I've opened a new pull request, #77, to work on those changes. Once the pull request is ready, I'll request review from you. |
* Initial plan * fix: update error messages to use PAWS_MAX_IMAGE_SIZE instead of FORGE_MAX_IMAGE_SIZE Co-Authored-By: ForgeCode <noreply@forgecode.dev> Co-authored-by: manthanabc <48511543+manthanabc@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: manthanabc <48511543+manthanabc@users.noreply.github.com>
* Initial plan * feat: restore read_image tool to sage and muse agents Co-Authored-By: ForgeCode <noreply@forgecode.dev> Co-authored-by: manthanabc <48511543+manthanabc@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: manthanabc <48511543+manthanabc@users.noreply.github.com>
* Initial plan * Remove accidentally committed echo file Co-Authored-By: ForgeCode <noreply@forgecode.dev> Co-authored-by: manthanabc <48511543+manthanabc@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: manthanabc <48511543+manthanabc@users.noreply.github.com>
No description provided.