Add integration tests for Graphite operations and missing attribution#1032
Open
jwiegley wants to merge 1 commit intojohnw/review-silent-errorsfrom
Open
Add integration tests for Graphite operations and missing attribution#1032jwiegley wants to merge 1 commit intojohnw/review-silent-errorsfrom
jwiegley wants to merge 1 commit intojohnw/review-silent-errorsfrom
Conversation
This was referenced Apr 9, 2026
Collaborator
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Apr 9, 2026
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
from
ed02bb9 to
9e18ac5
Compare
jwiegley
force-pushed
the
johnw/review-silent-errors
branch
2 times, most recently
from
93b3136 to
a4a730d
Compare
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
2 times, most recently
from
98654e8 to
68438da
Compare
jwiegley
force-pushed
the
johnw/review-silent-errors
branch
2 times, most recently
from
1338d42 to
a09eb0b
Compare
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
from
68438da to
99815bc
Compare
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
from
99815bc to
b1dbda4
Compare
jwiegley
force-pushed
the
johnw/review-silent-errors
branch
2 times, most recently
from
90c6d77 to
7c12fb9
Compare
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
from
b1dbda4 to
c51217f
Compare
jwiegley
force-pushed
the
johnw/review-silent-errors
branch
from
7c12fb9 to
b3492b6
Compare
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
from
c51217f to
a0ba1b3
Compare
New test suites covering two important scenarios: simulating Graphite CLI git operations (stack creation, rebase, amend workflows) and reproducing the intermittent bug where attribution incorrectly reports 100% human authorship. Adds allow_daemon_errors flag to test harness for controlled daemon failure testing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jwiegley
force-pushed
the
johnw/review-integration-tests
branch
from
a0ba1b3 to
6a82d28
Compare
jwiegley
force-pushed
the
johnw/review-silent-errors
branch
from
b3492b6 to
1d44bd3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
New test suites covering two important scenarios: simulating Graphite
CLI git operations (stack creation, rebase, amend workflows) and
reproducing the intermittent bug where attribution incorrectly reports
100% human authorship. Adds allow_daemon_errors flag to test harness
for controlled daemon failure testing.
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com