Repository navigation
Conversation
The comment JSON was assembled by string interpolation, with only double quotes stripped. Backslashes made the payload invalid (C:\Users, \d+, a trailing \), so the comment was rejected, and \n or \u0022 were reinterpreted. Stripping quotes also removed them from PR titles, bodies and review text, including the quotes the lightweight comments put around the title. Let jq escape the body instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A task linked more than once, e.g. in a Markdown link whose text is also the URL, got one comment, tag and stage move per link. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The IGNORE_PROJECT_IDS check exited the whole run, so tasks linked after one in an ignored project were never processed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The H1-H3 rewrite in the PR body also turned comment lines in fenced code blocks (shell, Python, YAML...) into #### headings. Track fences with awk and only rewrite lines outside them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Nothing calls ensure::total_args, github::get_pr_number, github::get_repository_full_name or github::print_all_data. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
It called echoerr, which doesn't exist, so an empty variable failed with
"command not found" (exit 127), and an unset one hit set -u ("unbound
variable") before the check ran. Also check GITHUB_EVENT_NAME, which the
action relies on.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Unsupported events looked up every linked task before giving up. Decide on the handler first, so they make no API calls. The next commit relies on this so that failed lookups can't fail runs for events the action ignores. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
curl ran without --fail, so HTTP errors (a revoked token, a task the token can't see, a Teamwork outage) were only logged and the run still passed. A failed task lookup also carried on with "null" as the project id. Network errors, on the other hand, aborted the run halfway, and a hung API held the runner until the job timed out. Pass --fail-with-body and --max-time 30, log what failed, carry on with the other steps and tasks, then exit 1 if anything failed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 20, 2026
seanogdev
approved these changes
Oct 20, 2026
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
2 participants
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.
Stack: 1. #75 tests → 2. #76 bug fixes → 3. #77 link matching → 4. #78 secrets → 5. #79 test mode removal
Summary
Fixes one bug per commit: comments with quotes or backslashes were mangled or rejected, tasks linked twice were updated twice, an ignored project stopped the remaining tasks,
#comments in code blocks became headings, and the required-variable check crashed. Failed Teamwork requests (wrong token, inaccessible task, outage) now fail the run instead of passing silently, and ignored events no longer call the API. It also removes four unused functions. Merge #77 right after this one: until then, text that only looks like a task link (subtasks/5) gets a 404 and fails the run.Evidence
Snapshot changes from
./tests/run.sh --updateat each commit:opened-lightweight: the title's quotes are stripped and the\tinC:\tempbecame a tab.opened,closed-*,review-approved*: multi-line comments are invalid JSON (-> 400). The mock is stricter than Teamwork about raw newlines; backslashes break Teamwork too.After:
-> 201with the full body.task-linksprocesses task 1001 three times. After: once.ignored-projectstops at the ignored task. After: task 1002 is commented on.openedturns# configure the "client" secretinside a code block into a####heading. After: left alone.unsupported-eventlooks the task up. After: no requests.wrong-token:exit code: 0, carrying on after the 401. After:GITHUB_REPOSITORYfails withechoerr: command not found(exit 127); an unset one with!1: unbound variable. After:The env variable GITHUB_REPOSITORY is required.(exit 1), checked by hand against the image.GITHUB_EVENT_NAMEis now checked too.🤖 Generated with Claude Code