Commit 6d1e842
committed
fix(bots): revert poetry lock/pyproject churn before author/followup publish
The engineer-bot author run on issue #860 failed at "Open / update fix PR":
publish's leftover safety-net found poetry.lock + pyproject.toml unstaged and
not in the agent's touched_files, so it refused to open the PR — even though the
agent's fix + tests were correct (494 passed).
Root cause: the shared setup-poetry action runs `poetry lock` (to reconcile the
lock with the internal JFrog source it injects), which REWRITES the tracked
poetry.lock / pyproject.toml in the working tree. That churn is harmless for the
5 real-CI consumers of the action (they commit nothing) but poisons the two
engineer-bot workflows, which publish a PR from the working tree.
Fix (engineer-bot side only — leave the shared action untouched): after the
Poetry setup, `git checkout -- poetry.lock pyproject.toml` to drop the churn once
the venv is built, so only the agent's edits remain in the tree at publish time.
Applied to both engineer-bot.yml (author) and engineer-bot-followup.yml.
Signed-off-by: eric-wang-1990 <e.wang@databricks.com>
Co-authored-by: Isaac1 parent 3e29a62 commit 6d1e842
2 files changed
Lines changed: 18 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
94 | 94 | | |
95 | 95 | | |
96 | 96 | | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
97 | 106 | | |
98 | 107 | | |
99 | 108 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
81 | 81 | | |
82 | 82 | | |
83 | 83 | | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
84 | 93 | | |
85 | 94 | | |
86 | 95 | | |
| |||
0 commit comments