Skip to content

fix(feedback): enforce anonymous submissions#1528

Merged
nwparker merged 1 commit intomainfrom
nwparker/ensure-anon-is-anon
May 7, 2026
Merged

fix(feedback): enforce anonymous submissions#1528
nwparker merged 1 commit intomainfrom
nwparker/ensure-anon-is-anon

Conversation

@nwparker
Copy link
Copy Markdown
Contributor

@nwparker nwparker commented May 7, 2026

Summary

  • make feedback anonymity an explicit IPC flag and allow-list the backend POST body
  • remove anonymous contact fields so checked anonymous never sends identity/contact data upstream
  • add regression coverage for stale identity/contact fields in anonymous feedback payloads
  • unblock current oxlint config parsing and replace the rich markdown boundary reset with derived state so full lint can run

Verification

  • pnpm exec vitest run --config config/vitest.config.ts src/main/ipc/feedback.test.ts
  • pnpm run tc:node
  • pnpm run tc:web
  • pnpm lint
  • pnpm exec oxfmt --check .oxlintrc.json src/main/ipc/feedback.ts src/main/ipc/feedback.test.ts src/preload/index.ts src/preload/api-types.ts src/renderer/src/components/sidebar/SidebarToolbar.tsx src/renderer/src/components/editor/RichMarkdownErrorBoundary.tsx

@nwparker nwparker force-pushed the nwparker/ensure-anon-is-anon branch from e7029cc to 43b126c Compare May 7, 2026 06:15
@nwparker nwparker force-pushed the nwparker/ensure-anon-is-anon branch from 43b126c to 3919a5b Compare May 7, 2026 06:21
@nwparker nwparker merged commit 43a2589 into main May 7, 2026
2 checks passed
@nwparker nwparker deleted the nwparker/ensure-anon-is-anon branch May 7, 2026 06:24
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