Repository navigation
Remove obsolete backend and frontend code - #80
Conversation
PR reader API performanceResult: INCONCLUSIVE 2 inconclusive · 6 within threshold. Zero request failures across all six runs. Base Three independent runs per revision, each on a fresh GitHub-hosted ARM64 runner. 16 users, 60 seconds/run, 1,000 synthetic articles, cache off, five PgBouncer slots. Latency comparisonAll latency values are in milliseconds; lower is better. Values are medians of three runs.
Additional latency and throughput metricsEach cell shows base → PR. Latencies are in milliseconds.
Individual runs
How to interpret this reportRegressed / Improved: a p95 regression needs >20% and >20 ms increase in the medians and at least two head samples versus the base median. Improvements use the inverse threshold. Within threshold means the change did not meet both thresholds. Inconclusive: a within-revision p95 range >35% of its median and >20 ms is too noisy for a confident latency verdict. Inconclusive latency comparisons are non-blocking: the check passes without claiming an improvement. Confirmed regressions and test/report errors fail the PR gate. Throughput is descriptive: paced users do not measure maximum capacity. This uncached public API test does not establish production health, browser performance or worker capacity. Separate runners can differ in hardware or host load; repetitions and noise checks reduce but cannot eliminate that uncertainty. Full HTML/CSV reports and logs · attempt 1 |
Code Review ✅ Approved🟡 Medium risk · Extension filters now depend on feed-options, changing behavior on older servers. Removes obsolete backend helpers ( OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |
Backend unit coveragePASS · minimum combined coverage 67.60%. Tests: 3007 passed, 0 failed, 0 skipped. Source files: 273.
Unit coverage includes every backend workspace. PostgreSQL/Redis integration tests run separately. Commit: 9de7974 · HTML, JSON, XML reports and job logs · attempt 1 |
|



What changes and why?
Remove unused backend and frontend code left behind by the current submission, overview and reader flows. Extensions also stop handling failed feed-options requests by fetching the entire source catalog and inventing content/language choices. They now use the web reader's existing behavior: successfully loaded articles remain readable, contextual filters return when the options route recovers.
A reference audit across tracked source, tests, scripts and documentation found no callers, registrations or routes for these internal exports:
job_display_status, the orphanSourceSubmissionOutschema, and the unusedretract_automatic_topichelperOverviewDetails,JobOutcomesChart, the serverinitialOverviewloader, andisResourcemcpServerGuideconstant/api/v1/feed/optionsThe active topic-correction repair operation remains covered by PostgreSQL regression tests. Existing overview panels and charts continue to render through their current loaders.
Retained compatibility code still serves persisted profile settings/links, saved reader/admin URLs, historical migrations and deployed CLI commands. A read-only production check confirmed
DEVFEED_WORKER_QUEUE=analysis, so that worker-group alias remains. This cleanup is isolated from the other open security, performance, coverage and backend-architecture PRs.Validation
Updated against master after #79 and the separate runtime-fixture fix #88, following the sequential merge queue. Conflict resolution preserves shared
DeliveryPolicyand both extension options-error and pending/failed article-feed fixtures.CI required, all three CodeQL analyses, Sonar, dependency/secret/workflow checks, populated upgrades, real API/admin journeys, actual web/Chrome/Edge browser parity, performance comparison and all ARM64 image build/scan/smoke checks.Rollout notes
The extension’s contextual filters require
/api/v1/feed/options, which exists in the current web app. Older servers without that route continue to display successfully loaded articles, with unavailable contextual filter choices omitted. Refresh the tab after server recovery to restore the choices.No database migration or configuration change is required. Website deployment and extension-store publication remain pending release. Merge this updated PR after #88 is merged and all current-head checks pass, then continue #81–#84 one at a time.
Ready for review