Skip to content

refactor: remove the unused release-feed widget (FeedItems.tsx) and its orphan modules - #1567

Open
hivecommons-hive[bot] wants to merge 1 commit into
mainfrom
arch/refactor-remove-dead-feeditems
Open

hivecommons-hive[bot] wants to merge 1 commit into
mainfrom
arch/refactor-remove-dead-feeditems

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Summary: This deletes about 2,000 lines of website code for an old release-feed widget that no page shows any more. Nothing on the site changes for visitors. It only removes code that looked live and kept getting edited by mistake.

What I need from you: Review and lift the hold if you agree.


Refactor

Removes the files that an import-graph walk over the repo finds unreachable from any page, theme override, doc, blog post, config or script:

  • src/components/FeedItems.tsx (no importer)
  • src/components/FeedItems.module.css, src/utils/sbomRelease.ts (imported only by FeedItems.tsx)
  • src/config/packageConfig.ts (no importer)
  • src/lib/sbom-feed-adapter.ts (no importer) and src/lib/sbom-adapter.ts (imported only by sbom-feed-adapter.ts)
  • src/types/theme.d.ts: the ambient declaration for @theme/useStoredFeed. Nothing provides that module any more, so this shim was the only reason FeedItems.tsx type-checked.

It also repoints three stale references from FeedItems to the live consumer of the SBOM cache-key format, sbomKeyForRelease() in src/utils/firehoseHelpers.ts (used by FirehoseFeed.tsx): comments in src/types/sbom.ts and scripts/lib/sbom/parser.js, and one test name in scripts/sbom-parser.test.js. The SBOM cache data and its fetch pipeline are unchanged, because FirehoseFeed.tsx and ImageProductSection.tsx still read them.

Verification

  • npx tsc --noEmit: same 6 errors as origin/main, all Cannot find module '@site/static/data/*.json' for generated data that fetch-data produces. No new errors.
  • npx eslint src scripts: 0 errors.
  • node --test "scripts/**/*.test.js": 1306 pass, 0 fail.
  • grep for FeedItems.tsx|sbomRelease|sbom-feed-adapter|packageConfig|useStoredFeed over src/, scripts/, docs/: no remaining references.

Closes #1566


Filed by architect agent (ACMM L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=architect backend=copilot model=claude-opus-5.5 copilot=1.0.88

FeedItems.tsx has no importer, and its @theme/useStoredFeed provider no
longer exists; only the ambient shim in src/types/theme.d.ts let it type
check. Remove it together with the modules reachable only from it
(FeedItems.module.css, utils/sbomRelease.ts, config/packageConfig.ts,
lib/sbom-feed-adapter.ts, lib/sbom-adapter.ts) and the shim, and point
the SBOM cache-key comments at the live consumer, sbomKeyForRelease()
in utils/firehoseHelpers.ts.

Closes #1566

Signed-off-by: architect <280983584+hivecommons-hive[bot]@users.noreply.github.com>
@hivecommons-hive
hivecommons-hive Bot requested a review from castrojo as a code owner October 22, 2026 02:02
@hivecommons-hive hivecommons-hive Bot added the hold Work is intentionally paused. label Oct 22, 2026
@hivecommons-hive

Copy link
Copy Markdown
Contributor Author

Important

Held for human review by the hive's ACMM level gate.

This PR was opened by the "architect" agent while Hive policy required a human checkpoint for that agent. Agent PRs are held at ACMM L3–L5; an outreach PR that targets a repository this hive does not manage is held at every level because it publishes project-facing communication into someone else's project.

Hive will keep the hold label until a human removes it. Operators can make a deliberate one-off release during an ACMM level change with release_level_holds=true, but level changes never release this hold automatically.

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from correctness, security, intent-alignment, style, docs-currency.

Confidence: 5/5 (safe)

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Work is intentionally paused.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[architect] Remove the unused release-feed widget code that no page shows any more

0 participants