Click Path Audit

作者 affaan-mef648e01899ba3e8dc6371642deaaf64b4477775无许可证275K 个星标收录于 2026年10月9日更新于 2026年10月9日仓库4天前更新

Trace every user-facing button/touchpoint through its full state change sequence to find bugs where functions individually work but cancel each other out, produce wrong final state, or leave the UI in an inconsistent state. Use when: systematic debugging found no bugs but users report broken buttons, or after any major refactor touching shared state stores.

仅含说明Software Development

AI 生成的概览

通过追踪状态变化审计界面点击处理程序，找出最终状态与按钮标签不符的问题。

功能: 提供一套结构化流程来审计交互式界面触点：先梳理每个状态存储操作及其设置或重置的字段，再按顺序追踪每个处理程序中的调用。它会检查六类缺陷模式，包括顺序撤销、异步竞态、过期闭包、缺失状态转换、死路径以及 useEffect 干扰。最终产出包含严重程度、触点位置、调用追踪、预期与实际行为以及修复建议的报告。
适用场景: 适用于系统化调试未发现缺陷但用户反馈按钮无响应或行为异常的情况，也适用于重构共享状态存储之后。它还用于发布前对关键用户流程的检查。
运行要求: 仅为说明文档，不含脚本。需要访问目标应用的源代码，包括组件处理程序和状态存储定义。

/click-path-audit — Behavioural Flow Audit

Find bugs that static code reading misses: state interaction side effects, race conditions between sequential calls, and handlers that silently undo each other.

The Problem This Solves

Traditional debugging checks:

Does the function exist? (missing wiring)
Does it crash? (runtime errors)
Does it return the right type? (data flow)

But it does NOT check:

Does the final UI state match what the button label promises?
Does function B silently undo what function A just did?
Does shared state (Zustand/Redux/context) have side effects that cancel the intended action?

Real example: A "New Email" button called setComposeMode(true) then selectThread(null). Both worked individually. But selectThread had a side effect resetting composeMode: false. The button did nothing. 54 bugs were found by systematic debugging — this one was missed.

How It Works

For EVERY interactive touchpoint in the target area:

1. IDENTIFY the handler (onClick, onSubmit, onChange, etc.)2. TRACE every function call in the handler, IN ORDER3. For EACH function call:   a. What state does it READ?   b. What state does it WRITE?   c. Does it have SIDE EFFECTS on shared state?   d. Does it reset/clear any state as a side effect?4. CHECK: Does any later call UNDO a state change from an earlier call?5. CHECK: Is the FINAL state what the user expects from the button label?6. CHECK: Are there race conditions (async calls that resolve in wrong order)?

1. IDENTIFY the handler (onClick, onSubmit, onChange, etc.)2. TRACE every function call in the handler, IN ORDER3. For EACH function call:   a. What state does it READ?   b. What state does it WRITE?   c. Does it have SIDE EFFECTS on shared state?   d. Does it reset/clear any state as a side effect?4. CHECK: Does any later call UNDO a state change from an earlier call?5. CHECK: Is the FINAL state what the user expects from the button label?6. CHECK: Are there race conditions (async calls that resolve in wrong order)?

Execution Steps

Step 1: Map State Stores

Before auditing any touchpoint, build a side-effect map of every state store action:

For each Zustand store / React context in scope:  For each action/setter:    - What fields does it set?    - Does it RESET other fields as a side effect?    - Document: actionName → {sets: [...], resets: [...]}

For each Zustand store / React context in scope:  For each action/setter:    - What fields does it set?    - Does it RESET other fields as a side effect?    - Document: actionName → {sets: [...], resets: [...]}

This is the critical reference. The "New Email" bug was invisible without knowing that selectThread resets composeMode.

Output format:

STORE: emailStore  setComposeMode(bool) → sets: {composeMode}  selectThread(thread|null) → sets: {selectedThread, selectedThreadId, messages, drafts, selectedDraft, summary} RESETS: {composeMode: false, composeData: null, redraftOpen: false}  setDraftGenerating(bool) → sets: {draftGenerating}  ...
DANGEROUS RESETS (actions that clear state they don't own):  selectThread → resets composeMode (owned by setComposeMode)  reset → resets everything

STORE: emailStore  setComposeMode(bool) → sets: {composeMode}  selectThread(thread|null) → sets: {selectedThread, selectedThreadId, messages, drafts, selectedDraft, summary} RESETS: {composeMode: false, composeData: null, redraftOpen: false}  setDraftGenerating(bool) → sets: {draftGenerating}  ...
DANGEROUS RESETS (actions that clear state they don't own):  selectThread → resets composeMode (owned by setComposeMode)  reset → resets everything

Step 2: Audit Each Touchpoint

For each button/toggle/form submit in the target area:

TOUCHPOINT: [Button label] in [Component:line]  HANDLER: onClick → {    call 1: functionA() → sets {X: true}    call 2: functionB() → sets {Y: null} RESETS {X: false}  ← CONFLICT  }  EXPECTED: User sees [description of what button label promises]  ACTUAL: X is false because functionB reset it  VERDICT: BUG — [description]

TOUCHPOINT: [Button label] in [Component:line]  HANDLER: onClick → {    call 1: functionA() → sets {X: true}    call 2: functionB() → sets {Y: null} RESETS {X: false}  ← CONFLICT  }  EXPECTED: User sees [description of what button label promises]  ACTUAL: X is false because functionB reset it  VERDICT: BUG — [description]

Check each of these bug patterns:

Pattern 1: Sequential Undo

handler() {  setState_A(true)     // sets X = true  setState_B(null)     // side effect: resets X = false}// Result: X is false. First call was pointless.

handler() {  setState_A(true)     // sets X = true  setState_B(null)     // side effect: resets X = false}// Result: X is false. First call was pointless.

Pattern 2: Async Race

handler() {  fetchA().then(() => setState({ loading: false }))  fetchB().then(() => setState({ loading: true }))}// Result: final loading state depends on which resolves first

handler() {  fetchA().then(() => setState({ loading: false }))  fetchB().then(() => setState({ loading: true }))}// Result: final loading state depends on which resolves first

Pattern 3: Stale Closure

const [count, setCount] = useState(0)const handler = useCallback(() => {  setCount(count + 1)  // captures stale count  setCount(count + 1)  // same stale count — increments by 1, not 2}, [count])

const [count, setCount] = useState(0)const handler = useCallback(() => {  setCount(count + 1)  // captures stale count  setCount(count + 1)  // same stale count — increments by 1, not 2}, [count])

Pattern 4: Missing State Transition

// Button says "Save" but handler only validates, never actually saves// Button says "Delete" but handler sets a flag without calling the API// Button says "Send" but the API endpoint is removed/broken

// Button says "Save" but handler only validates, never actually saves// Button says "Delete" but handler sets a flag without calling the API// Button says "Send" but the API endpoint is removed/broken

Pattern 5: Conditional Dead Path

handler() {  if (someState) {        // someState is ALWAYS false at this point    doTheActualThing()    // never reached  }}

handler() {  if (someState) {        // someState is ALWAYS false at this point    doTheActualThing()    // never reached  }}

Pattern 6: useEffect Interference

// Button sets stateX = true// A useEffect watches stateX and resets it to false// User sees nothing happen

// Button sets stateX = true// A useEffect watches stateX and resets it to false// User sees nothing happen

Step 3: Report

For each bug found:

CLICK-PATH-NNN: [severity: CRITICAL/HIGH/MEDIUM/LOW]  Touchpoint: [Button label] in [file:line]  Pattern: [Sequential Undo / Async Race / Stale Closure / Missing Transition / Dead Path / useEffect Interference]  Handler: [function name or inline]  Trace:    1. [call] → sets {field: value}    2. [call] → RESETS {field: value}  ← CONFLICT  Expected: [what user expects]  Actual: [what actually happens]  Fix: [specific fix]

CLICK-PATH-NNN: [severity: CRITICAL/HIGH/MEDIUM/LOW]  Touchpoint: [Button label] in [file:line]  Pattern: [Sequential Undo / Async Race / Stale Closure / Missing Transition / Dead Path / useEffect Interference]  Handler: [function name or inline]  Trace:    1. [call] → sets {field: value}    2. [call] → RESETS {field: value}  ← CONFLICT  Expected: [what user expects]  Actual: [what actually happens]  Fix: [specific fix]

Scope Control

This audit is expensive. Scope it appropriately:

Full app audit: Use when launching or after major refactor. Launch parallel agents per page.
Single page audit: Use after building a new page or after a user reports a broken button.
Store-focused audit: Use after modifying a Zustand store — audit all consumers of the changed actions.

Recommended agent split for full app:

Agent 1: Map ALL state stores (Step 1) — this is shared context for all other agentsAgent 2: Dashboard (Tasks, Notes, Journal, Ideas)Agent 3: Chat (DanteChatColumn, JustChatPage)Agent 4: Emails (ThreadList, DraftArea, EmailsPage)Agent 5: Projects (ProjectsPage, ProjectOverviewTab, NewProjectWizard)Agent 6: CRM (all sub-tabs)Agent 7: Profile, Settings, Vault, NotificationsAgent 8: Management Suite (all pages)

Agent 1: Map ALL state stores (Step 1) — this is shared context for all other agentsAgent 2: Dashboard (Tasks, Notes, Journal, Ideas)Agent 3: Chat (DanteChatColumn, JustChatPage)Agent 4: Emails (ThreadList, DraftArea, EmailsPage)Agent 5: Projects (ProjectsPage, ProjectOverviewTab, NewProjectWizard)Agent 6: CRM (all sub-tabs)Agent 7: Profile, Settings, Vault, NotificationsAgent 8: Management Suite (all pages)

Agent 1 MUST complete first. Its output is input for all other agents.

When to Use

After systematic debugging finds "no bugs" but users report broken UI
After modifying any Zustand store action (check all callers)
After any refactor that touches shared state
Before release, on critical user flows
When a button "does nothing" — this is THE tool for that

When NOT to Use

For API-level bugs (wrong response shape, missing endpoint) — use systematic-debugging
For styling/layout issues — visual inspection
For performance issues — profiling tools

Integration with Other Skills

Run AFTER /superpowers:systematic-debugging (which finds the other 54 bug types)
Run BEFORE /superpowers:verification-before-completion (which verifies fixes work)
Feeds into /superpowers:test-driven-development — every bug found here should get a test

Example: The Bug That Inspired This Skill

ThreadList.tsx "New Email" button:

onClick={() => {  useEmailStore.getState().setComposeMode(true)   // ✓ sets composeMode = true  useEmailStore.getState().selectThread(null)      // ✗ RESETS composeMode = false}}

onClick={() => {  useEmailStore.getState().setComposeMode(true)   // ✓ sets composeMode = true  useEmailStore.getState().selectThread(null)      // ✗ RESETS composeMode = false}}

Store definition:

selectThread: (thread) => set({  selectedThread: thread,  selectedThreadId: thread?.id ?? null,  messages: [],  drafts: [],  selectedDraft: null,  summary: null,  composeMode: false,     // ← THIS silent reset killed the button  composeData: null,  redraftOpen: false,})

selectThread: (thread) => set({  selectedThread: thread,  selectedThreadId: thread?.id ?? null,  messages: [],  drafts: [],  selectedDraft: null,  summary: null,  composeMode: false,     // ← THIS silent reset killed the button  composeData: null,  redraftOpen: false,})

Systematic debugging missed it because:

The button has an onClick handler (not dead)
Both functions exist (no missing wiring)
Neither function crashes (no runtime error)
The data types are correct (no type mismatch)

Click-path audit catches it because:

Step 1 maps selectThread resets composeMode
Step 2 traces the handler: call 1 sets true, call 2 resets false
Verdict: Sequential Undo — final state contradicts button intent

来源与署名

来源：affaan-m/ECC位于skills/click-path-audit提交ef648e0

许可证：无许可证

内容归原作者所有。SourceWeft 从公开仓库中收录这些内容。

举报或申请下架

Click Path Audit

/click-path-audit — Behavioural Flow Audit

The Problem This Solves

How It Works

Execution Steps

Step 1: Map State Stores

Step 2: Audit Each Touchpoint

Pattern 1: Sequential Undo

Pattern 2: Async Race

Pattern 3: Stale Closure

Pattern 4: Missing State Transition

Pattern 5: Conditional Dead Path

Pattern 6: useEffect Interference

Step 3: Report

Scope Control

Recommended agent split for full app:

When to Use

When NOT to Use

Integration with Other Skills

Example: The Bug That Inspired This Skill

来源与署名

更多来自 affaan-m/ECC 的技能

Unified Notifications Ops

Verification Loop

Ui To Vue

Unified Memory

Visa Doc Translate

Team Builder

更多Software Development技能

Playground

M5 Onboard

Cardputer Buddy

Web Design Guidelines

Vercel React Native Skills

Vercel React Best Practices