Skip to content

docs(skills): add Symphony ticket requirements - #14

Merged
big-mon merged 6 commits into
mainfrom
codex/symphony-ticket-skill
Sep 24, 2026
Merged

big-mon merged 6 commits into
mainfrom
codex/symphony-ticket-skill

Conversation

@big-mon

@big-mon big-mon commented Sep 24, 2026 •

Copy link
Copy Markdown
Owner

Context

Linear tickets need the monitored project and a valid Repository child label before Symphony can process them. Ticket authors need reusable requirements that respect each deployment's optional routing conditions.

TL;DR

Add a concise skill for preparing and verifying Linear tickets before handing them to Symphony.

Summary

  • Require the monitored project, one valid Repository child label, essential content, and dependencies.
  • Resolve optional routing conditions per deployment; require assignee readback only when a filter is configured.
  • Prepare unstarted issues in the pre-execution holding state (Backlog here), then verify and activate in Todo.
  • Leave active, review-stage, and unknown issues unchanged; route review-stage requirement changes through rework.
  • With unresolved blockers, use active Todo or wait outside active states until all blockers are terminal.

Alternatives

  • Keep ordinary Linear operations with existing tools; hold ambiguous creates instead of adding a custom retry protocol.
  • Keep scheduler coordination outside ticket authoring; a status snapshot is not a reservation against dispatch.
  • Do not require assignment where Symphony has no assignee filter.

Test Plan

  • Skill Creator quick_validate.py and git diff --check passed; only SKILL.md changed.
  • Walked through new and unstarted Backlog issues with separate dependency writes before Todo activation.
  • Verified active queued, running, Human Review, terminal, and unknown issues take handoff without mutation.
  • Checked unresolved and unknown blockers with Todo enabled or absent, and terminal blockers.
  • Checked registration-only requests, ambiguous creation outcomes, and initial-work versus lifecycle states.
  • Checked no-filter, explicit assignee, and me cases.
  • Standalone pr_body.check passed.
  • Full make all is not applicable: only an allowlisted skill procedure changed.

Define the project, Repository label, execution and dependency requirements
for Linear tickets handed to Symphony. Include essential issue content and
verification after registration without duplicating ordinary Linear usage.

Keep Personal Symphony explicit for this environment while resolving IDs
and other environments from their configuration.

Validation: skill quick validation, procedure review, diff check, and
standalone PR body validation passed. Full Elixir checks are not applicable
for this skill-only change.

Co-authored-by: Codex <codex@openai.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-24T14:37:47.464243Z a29bdab New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ebaa09b03e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Comment thread .codex/skills/symphony-ticket/SKILL.md
Trigger the skill for creation and updates as well as requirement checks.
Keep new work outside active states until its fields and dependencies are
saved and verified, then activate and read back the final state.

Apply optional routing requirements only when configured. Verify assignee
IDs when a filter exists, resolving me against Symphony's identity rather
than the ticket author's identity. Preserve already-running issue states.

Validation: skill validator, diff check, standalone PR body lint, and
procedure walkthroughs passed. Full Elixir validation is not applicable
for this skill-only change.

Co-authored-by: Codex <codex@openai.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c630780b46

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Stage new issues and queued updates in a non-active, non-terminal state
before editing fields, and verify the saved issue before activation.
Coordinate with execution for running or indeterminate issues instead of
automatically resetting their state.

Document that the standard Linear client checks blockers only in Todo.
Use Todo when active, or retain blocked work outside active states until
all blockers are terminal. Keep optional assignee requirements conditional.

Validation: skill validator, diff check, standalone PR body validation,
and walkthroughs for queued updates, running/unknown states, and blocker
routing passed. Full Elixir validation does not apply to this skill-only
change.

Co-authored-by: Codex <codex@openai.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b78c19e6f9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Limit ticket creation and updates to inactive, non-terminal issues.
Leave active and unknown-state issues unchanged and hand them to the
execution-side operating procedure, even when they appear queued.

A runtime snapshot cannot prevent a scheduler from claiming an issue
before a later state change. Remove that check-then-reset procedure
instead of adding scheduler pause or reservation machinery to this skill.

Validation: skill validator, diff check, standalone PR body lint, and
walkthroughs of inactive, active queued, running, and unknown-state issues
passed. Full Elixir validation does not apply to this skill-only change.

Co-authored-by: Codex <codex@openai.com>
@big-mon

big-mon commented Sep 24, 2026

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 71fa620f6c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .codex/skills/symphony-ticket/SKILL.md
Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Name Todo as this deployment's initial-work state and exclude approval or
rework states from new execution. Other deployments use their configured
initial-work state while preserving the dependency condition.

Do not repeat a create with an unknown outcome. Hold execution until the
existing issue is identified instead of adding a creation retry protocol.

Validation: skill validator, diff check, standalone PR body validation,
and walkthroughs for lifecycle states and ambiguous creates passed.
Full Elixir validation does not apply to this skill-only change.

Co-authored-by: Codex <codex@openai.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d2bba4d82d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .codex/skills/symphony-ticket/SKILL.md Outdated
Specify Backlog as this deployment's pre-execution holding state and use
an equivalent initial holding state elsewhere. Exclude Human Review and
other review or approval stages even when they are inactive.

Hand existing issues outside the unstarted holding state to execution
without mutation. Review-stage requirement changes follow rework instead
of invalidating the requirements of a review-ready PR in place.

Validation: skill validator, diff check, standalone PR body lint, and
walkthroughs of Backlog, active, Human Review, terminal, and unknown issues
passed. Full Elixir validation is not applicable to a skill-only change.

Co-authored-by: Codex <codex@openai.com>
@big-mon
big-mon merged commit 956b8c8 into main Sep 24, 2026
3 checks passed
@big-mon
big-mon deleted the codex/symphony-ticket-skill branch September 24, 2026 14:52
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