Skip to content

Dev#129

Merged
dcloud347 merged 4 commits intomainfrom
dev
Sep 11, 2025
Merged

Dev#129
dcloud347 merged 4 commits intomainfrom
dev

Conversation

@dcloud347
Copy link
Contributor

This pull request introduces persistent shell support for Docker containers in the BaseContainer class, enabling command execution with maintained context and improved error handling. It also refactors several LangGraph nodes to allow more flexible keying and patch handling, improving modularity and configurability across the pipeline. Additionally, a new DockerException class is introduced for clearer exception handling related to Docker operations.

Docker Container Management Enhancements:

  • Added persistent shell support to BaseContainer using pexpect, including initialization, restart logic, and cleanup; command execution now uses this shell for better context and output handling. [1] [2] [3] [4] [5] [6] [7] [8]
  • Introduced DockerException as a base class for Docker-related errors for clearer error reporting.

LangGraph Node Refactors and Flexibility:

  • Refactored FinalPatchSelectionNode to use configurable keys for candidate patches, allowing it to operate on arbitrary patch lists in the state dictionary. [1] [2] [3] [4]
  • Updated PatchNormalizationNode to accept input and output keys as parameters, enabling greater flexibility in patch deduplication and normalization stages. [1] [2] [3] [4]
  • Enhanced GetPassRegressionTestPatchSubgraphNode to allow configurable return keys and patch formats, improving integration with downstream nodes. [1] [2] [3]

Bug Verification and Test Node Adjustments:

  • Adjusted logic in bug verification and regression test nodes to better handle patch lists and output formats, ensuring correct propagation of candidate patches and logs. [1] [2]

Other Improvements:

  • Minor prompt and typo corrections for clarity in regression test instructions.

@dcloud347 dcloud347 merged commit 98396fc into main Sep 11, 2025
2 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.

1 participant