Skip to content

create-ticket: auto-add issues to Project #50 and link sub-issue parents - #656

Merged
rhyslutsky merged 3 commits into
feature/voice-selectionfrom
feature/654/create-ticket-project-autolink
Aug 4, 2026
Merged

create-ticket: auto-add issues to Project #50 and link sub-issue parents#656
rhyslutsky merged 3 commits into
feature/voice-selectionfrom
feature/654/create-ticket-project-autolink

Conversation

@rhyslutsky

Copy link
Copy Markdown
Collaborator

Summary

Ticket

Closes #654 — same non-default-branch caveat as #653/PR #655: will manually close after merge since closes #N won't auto-fire against feature/voice-selection.

Type of Change

  • New feature
  • Bug fix
  • Documentation
  • Refactor
  • Tests
  • CI/CD

Testing

  • Unit tests added/updated — N/A, this is a skill doc, not app code
  • Integration tests added/updated — N/A, same reason
  • Manual testing performed — see Documentation/work-log/654-create-ticket-project-autolink.md for the full verification writeup, including the one gap found: createLinkedBranch needs repo write access distinct from the project scope (confirmed live via a FORBIDDEN response before this PR's write access was granted)

Checklist

  • Tests pass locally (./gradlew testDebug) — N/A, no app code (app/) touched
  • No API keys or secrets in code — all mutations shell out to gh api graphql, reusing the machine's existing gh auth login
  • CLAUDE.md updated (if new pattern introduced) — not updated; doesn't introduce an app-development pattern

See Documentation/work-log/654-create-ticket-project-autolink.md for the complete writeup.

New tickets now get added to the board (Status: Pre Backlog) instead of
staying invisible until someone remembers to do it manually. Part-of
sub-issues also get linked to their parent via GitHub's native addSubIssue
relationship, not just the prose reference, so the project's Parent
issue/Sub-issues progress fields actually reflect it.

Closes #654
…elopment

New tickets now get positioned at the front of the project's item order
(updateProjectV2ItemPosition, afterId omitted), which is sufficient to put
them first within their own Status group in the grouped board view.

Also documents createLinkedBranch for creating the working branch through
the issue's native Development link, so it's visible there from the start
of work rather than only once a PR exists. Confirmed live this needs repo
write access (FORBIDDEN), distinct from the project scope requirement for
the board-add mutations above it.
Live-tested against the just-opened PRs #655/#656: Closes #N does not
register as a Development link at all when the PR targets a non-default
branch (confirmed via closingIssuesReferences and willCloseTarget both
being empty/false), not just "won't auto-close on merge" as previously
assumed. createLinkedBranch must run first, before any branch of that name
is pushed, since there's no mutation to retroactively link an existing one.
@rhyslutsky rhyslutsky linked an issue Aug 4, 2026 that may be closed by this pull request
5 tasks

@troymyree-lang troymyree-lang left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving — clean and thorough. Nice verification discipline: dogfooding the project-add and Status steps on #654, confirming the read-side parent link is real (#630#613), and deliberately not fabricating a test sub-issue write to avoid polluting real timelines. The createLinkedBranch-first finding — that Closes #N doesn't even link under Development when the PR targets a non-default branch — is a genuinely useful catch.

Two non-blocking notes: the hardcoded project/field/option IDs are fine given you included the re-derive query, and the write path (addSubIssue / createLinkedBranch) is documented but unexercised until the next real ticket, so it's worth eyeballing the first ticket created via the updated skill to confirm that path works end to end.

@rhyslutsky
rhyslutsky merged commit cc8b0c7 into feature/voice-selection Aug 4, 2026
4 checks passed
@rhyslutsky
rhyslutsky deleted the feature/654/create-ticket-project-autolink branch August 4, 2026 20:24
rhyslutsky added a commit that referenced this pull request Aug 6, 2026
Per review on #655/#666: rather than relocating gh_fetch.py to
dq-documentation while still vendoring project-specific copies here, the
whole skill is now installed at ~/.claude/skills/github-throughput-forecast/
instead - available for any GitHub Projects v2 board, nothing committed
into any one repo's history.

Removes .claude/scripts/{gh_fetch,forecast,forecast-html}.py,
.claude/skills/throughput-forecast/, and the settings.json that existed
solely for their Bash allowlist. create-ticket (#654/#656) untouched -
separate concern, out of scope. No merged history reverted - this is a
normal forward delete commit.

PR #665 closed unmerged rather than revised (its fixes carried over into
the global install already).

Closes #666
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.

create-ticket: auto-add issues to Project #50 and link sub-issue parents

2 participants