Complete the Slack feedback cycle: read every thread and linked evidence, fix verified repo-owned issues, and reply in-thread with honest status, clarification questions, and release follow-up. Use when the user asks to address feedback and respond in the requested voice through the invoking user's connected Slack identity.
61
77%
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
Low
Low-risk findings worth noting
Fix and improve this skill with Tessl
tessl review fix ./.agents/skills/address-feedback-with-replies/SKILL.mdUse this workflow when the user wants feedback triaged, fixed, and answered in
Slack in one pass. Reply only in the requested scope and keep replies as
evidence-based as the code. Cluster identical symptoms under one Builder
thread and follow review-latest-feedback's single-owner reply rule.
This workflow handles clear bugs and concrete design/UX feedback about existing
repo-owned surfaces. In Slack sweeps, follow review-latest-feedback for
eligibility: do not exclude Design feedback because it is visual or subjective;
new capability requests still require the invoking identity's :upvote:.
Praise, status updates, merge or review requests, bot forwards, duplicates, and
other unrelated messages stay out of scope. Content remains with Alice unless
the user explicitly assigns it.
An :upvote: from the invoking identity promotes a new-capability request into
scope - that reaction is the product decision, so build the smallest version
rather than asking which variant is wanted. The upvote is the authorization —
do not wait for a second sign-off.
It does not transfer ownership: an upvoted Design or Content item still gets
built, with Sid or Alice named in the recap row so the mapped owner is not
surprised by a change in their area. Naming them is a courtesy, not a gate.
Either workflow may complete an upvoted improvement and record the terminal
Shipped disposition. Everything in this file about voice, evidence, and
verification applies to an upvoted item unchanged.
Even when invoked alone, this workflow asks at most three new clarification
questions per run across all threads, ranked by which answer would unblock a
safe fix.
If this workflow earlier added 👀 before recognizing an item was out of
scope, keep our eye and mark the unfixed item with 🎫. Record Skipped
with one brief status reply if the thread lacks it, not a question. If this
workflow already posted a mistaken reply, edit it to that disposition.
New messages must pass the clear-bug gate before any external write.
Use the disposition-specific reaction contract from review-latest-feedback:
add 👀 when claiming, ✅ for verified fixes, and 🎫 for items left
unfixed. Mixed threads get both status reactions. Reactions are never removed;
newer thread evidence determines the current disposition.
Follow review-latest-feedback for ownership and cluster status; a claimed
report keeps its 👀, and an unresolved eye cannot be taken over.
address-feedback, concurrent-agents, and verifying-changes first.new-branch when needed; carry only this task's
changes there.
Never reset, stash, or overwrite peer work.Every Slack interaction in this workflow - channel history, reactions, thread
replies and read-backs, permalinks, and Slack message/user/file metadata - uses
the connected Slack identity of the user who invoked the workflow. Verify that
identity before the first write and keep its stable { team_id, user_id }
tuple for all read-backs. Do not silently switch to a bot, another user's
connector, or a different workspace. Do not load or use SLACK_BOT_TOKEN.
Linked external artifacts are not Slack interactions. After extracting their reference from Slack, fetch them through the artifact's owning connector or public URL.
user_id; missing author data is unverified and cannot satisfy
the reply ledger. It must also match the target channel, timestamp, thread
parent, and workspace. A reaction read-back must include the invoking user
in the reaction's user list.Apply the shared address-feedback Choose the fix altitude gate before
reacting, editing code, or replying. It selects the smallest owning seam and
prevents one subjective report from becoming a global instruction.
For Slack sweeps, review-latest-feedback owns scope and reaction eligibility.
Honor its inclusion of concrete Design/UX feedback about existing surfaces and
the invoking user's assignment; do not override it with a generic UX exclusion.
New-capability requests still need the invoking identity's :upvote:. Once an
item is in scope, add 👀 before investigation or delegation.
During triage, follow the single-owner rule. After merge, follow
review-latest-feedback: reply once in every affected source thread, including
clusters.
A tracked clear bug or authorized upvoted improvement receives at most one
disposition per run. Verified locally, Built - live unverified,
Deployed - live unverified, Not reproducible - attempted, and In
progress retain 👀 after the report has been claimed.
Asked, Clarification needed, and Blocked on reporter retain 👀
after the report has been claimed.
Terminal dispositions: Fixed, Shipped, Live verified,
Open - no question, Resolved elsewhere, Skipped, Clustered, and
Abandoned - no answer in 4 days, each with required evidence and 👀; add
✅ only for verified fixes and 🎫 for unfixed items. An already-eyed
out-of-scope item keeps its eye and gets one concise Skipped reply if the
thread lacks that status.
Fixed closes the issue after a verified
source fix merges; publication, beta, and live verification follow separately.
In progress is an open ownership state. Do not send an interim progress
reply for work fixed in the same run. If work continues beyond this run, follow
review-latest-feedback and post one concrete status before the next sweep.
Continue or coordinate existing work instead of starting a duplicate. Never
use a vague update or replace verification. The next run must revisit this
state and resolve it to Fixed, Clarification needed, or evidence-backed
Open - no question when no safe fix or reproduction remains. Blocked,
not fixed yet, still needs a fix, and similar phrases are internal notes,
never a complete Slack reply. Open - no question is terminal with our eye
retained and no checkmark. A reply must state a verified resolution, concrete
active work, a needed question, or one terminal disposition with its reason;
never send a vague status alone. These are ledger states, not mandatory
headings: keep the reporter-facing wording natural instead of opening with the
robotic phrase “Clarification needed”. A
substantive diagnosis, fix, or in-progress ownership statement from someone in
the thread is not a reason to ask for clarification; verify it or continue the
existing handoff first.
Clarification needed is an open state, not a completed product fix. Asking
the question creates a standing obligation to come back for the answer. It is
the invoking identity's open disposition for the current cursor, not a terminal
closure; keep the eye while waiting. The next
review-latest-feedback run must re-read every thread it previously asked in
before scanning newer messages; when this workflow runs on its own, do the same
and act on the replies first.
That obligation expires after four days, standalone runs included: keep our
👀, add 🎫 for the unfixed item, and post Abandoned - no answer in 4
days once if the thread lacks that status. Ask nothing further; carry any
active bug forward.
In progress is also an open state. It records that the thread already has real ownership or an active fix, so the invoking identity must not ask the reporter to repeat the issue. Re-read it on the next run, verify the work, and replace the open state with Fixed when complete or Clarification needed only if a specific reporter or product input is still missing.
Treat a clarification reply as new evidence, not a new report. Re-read the thread and try the fix before asking anything else. An answer or explicit resolution from any participant is sufficient when it supplies the requested detail. A partial reply leaves the one existing request pending. Ask again only for one specific, non-repeating detail that still blocks a clear-bug fix; never ask for a subjective product choice or evidence already present. For an inaccessible artifact, request access or a replacement link.
This workflow inherits the hard cap from review-latest-feedback: at most
three new clarification questions per invocation across all threads. When run
alone, rank candidates by whether the answer would unblock a safe fix, handle
existing clarification requests first, and leave candidates below the cut
open without asking. Never turn the per-item question rule into an unbounded
batch.
There may be only one unanswered clarification request per thread. Before
posting, re-read the complete thread for an earlier question from this
workflow, the companion review-latest-feedback workflow, or @agent-native,
and check whether its exact requested detail has been semantically answered or
explicitly resolved anywhere in the thread. If it is still unresolved,
including after a partial or unrelated reply, keep that request as the sole
pending handoff and do not post another question. Once it is answered or
resolved, attempt the fix from the new evidence first; ask at most one new,
non-repeating question only if one specific required detail still blocks it.
address-feedback categorization and Fix-altitude
gate when choosing the disposition and owning seam. When the same underlying
issue appears in multiple threads, build one cluster checklist and drive one
Builder thread for that cluster; do not create separate Builder threads
unless the reports diverge in symptom, surface, or owner.👀 to
each clear bug and every actionable Design/UX item already classified in
scope, one thread at a time as it enters scope. Do not
batch reactions until after investigation, implementation, testing, or the
final Slack pass. If the reaction fails, stop and retry or report the
concrete Slack permission/API blocker before continuing the investigation.
Do not react to out-of-scope product requests, policy or informational
messages, bot forwards, status-only items, or non-repo-owned reports.
For an authorized upvoted improvement, perform and read back that same eye
reaction before investigation or delegation, then include it in the ledger.👀. Record clustered source links and disposition in the owner
thread or linked work; do not send duplicate replies:
review-latest-feedback for the post-merge reply:
name the fix, say it merged, and give the correct beta or package timing.
Never claim it is live without runtime proof.thread_ts through the same connected Slack identity. Do not
silently turn an authorized write into a draft. Re-read each thread
afterward to confirm the reply landed under the intended parent. Use the
exact parent timestamp as thread_ts; never reply to a search-result
timestamp or an adjacent thread. Before ending the run, mechanically
audit the reply ledger: for each claimed parent, record its reply timestamp
or owner-thread link, disposition, and eye state. A terminal state in the
thread or linked work needs no duplicate reply. Record Owned elsewhere
only when the latest update confirms another active owner; preserve foreign
eyes. Record non-owning Clustered rows in the owner thread or linked
work, and send no duplicate reply unless a distinct question or update needs
an answer. Record out-of-scope items with one Skipped status if missing.
If any participant replies after the post, re-read the entire thread again
before deciding whether to fix, close, or ask anything else.Write in the invoking user's voice and send from that same connected Slack identity:
ty for the feedback - (or thanks for the feedback -) before the
status. Do not open with agreed, valid request, ah, or a diagnosis.this was sent from a bot. so
future sweeps can rediscover it; historical replies may not contain the
marker and must still be found by the companion clarification search.ah, yeah, and good find can follow the thank-you when they fit; do not
force them into every reply. Prefer - over em dashes.valid request and stop, and do not say no ship timing yet as a dead end.
Implement the fix first. After its PR merges, follow
review-latest-feedback to report the fix and give the correct beta or
package timing. Do not promise beta timing before merge.review-latest-feedback; a merge is not live
proof.👀, add 🎫, record
Open - no question, and post that status once if the thread lacks it.✅ only for verified fixes and 🎫
to items left unfixed.A useful reply shape is:
ty for the feedback - [short plain-language status].
[if a merged app fix uses the beta publisher: the fix merged and should be
live on beta in the next few hours.]
[if the report is mixed: name each unfixed item and why; ask one targeted
question if needed, or say clear deferred work will be ticketed.]
[if in progress: we're already looking into this and will follow up once the
fix is verified.]
[if clarification is needed: if you can share the one missing detail, that
would help us investigate, such as a deck URL and/or request ID.]Keep it to one short paragraph whenever possible. Omit the release sentence only when the change is not complete; do not invent a ship date for an open item. For an unclear runtime report, inspect its run ID and linked app evidence first; ask for one missing detail only when those sources cannot adequately identify the failure.
After the final ship, return to the same threads and post a brief follow-up saying the fix is live only after verifying the live path internally. If the ship has not happened yet or live verification is unavailable, do not imply that the fix is already live. Omit commit, release, and live-path details from the posted follow-up unless the reporter asks for them.
git diff --check.address-feedback - classification and fix boundaries.concurrent-agents - shared-checkout coordination.verifying-changes - proof before claiming done.writing-agent-instructions - guidance quality and scope.a941a2e
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.