Skip to content

Harden feedback collector end to end#5

Merged
HugoMSFT merged 4 commits into
mainfrom
hugomsft-harden-feedback-collector
Jul 18, 2026
Merged

Harden feedback collector end to end#5
HugoMSFT merged 4 commits into
mainfrom
hugomsft-harden-feedback-collector

Conversation

@HugoMSFT

@HugoMSFT HugoMSFT commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • make SQLite the authoritative feedback store, preserve user edits across collection and Fabric synchronization, and add forward-safe migrations
  • secure credentials, request boundaries, remote access, CSV handling, browser rendering, and collector logging
  • replace fragile background state with durable cancellable jobs and tighten collection/Fabric error handling
  • correct Fabric token packing, canonical ID synchronization, existing-row updates, state conflict handling, and schema migration behavior
  • improve packaged-data persistence and legacy migration, remove obsolete integrations/docs, and refresh dependencies
  • prioritize SQL Server and Azure SQL sources and keywords; add public Hacker News and DEV Community collectors plus SQL-focused Stack Exchange and GitHub defaults
  • support multiple Reddit communities with normalized comma/newline input, saved-setting migration, validation, sort/time filters, and fair item-limit distribution
  • skip missing credential-dependent sources and isolate source failures so successful public-source results are retained
  • add cross-platform CI plus regression coverage for routes, persistence, security, jobs, Fabric SQL, collectors, frontend behavior, and packaging paths

Validation

  • python -m unittest discover -s tests -v — 88 tests pass
  • Python compilation, JavaScript parsing, and diff whitespace checks pass
  • live Hacker News, DEV Community, and Stack Overflow API smoke tests pass
  • live application collection completed with Hacker News and DEV Community results
  • PyInstaller distribution builds successfully
  • frozen Windows application responds over loopback, uses per-user application data, and excludes .env

HugoMSFT and others added 4 commits July 18, 2026 09:03
Make SQLite authoritative, secure runtime and browser boundaries, add durable jobs and Fabric synchronization, strengthen packaging, and expand regression coverage.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Build the expected path with the host path implementation so the Windows branch-selection test remains valid on Linux CI runners.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Prioritize SQL Server and Azure SQL across default sources and keywords, add Hacker News and DEV Community collectors, and isolate optional source failures so one integration cannot discard a run.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Add comma and newline separated subreddit configuration, normalize prefixes and URLs, migrate saved settings, and divide collection limits across communities.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@HugoMSFT
HugoMSFT merged commit 703eaab into main Jul 18, 2026
8 checks passed
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