Skip to content

chore: remove unused release/contributors fields from github query - #38

Open
detail-app[bot] wants to merge 1 commit into
devfrom
detail/dead-code/chore-remove-unused-release-contributors-fields-fr-d23b73
Open

detail-app[bot] wants to merge 1 commit into
devfrom
detail/dead-code/chore-remove-unused-release-contributors-fields-fr-d23b73

Conversation

@detail-app

@detail-app detail-app Bot commented Sep 15, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #

Type of change

  • Refactor / code improvement

What does this PR do?

Removes the dead release and contributors fields (and their supporting fetch + Link-header parsing) from the github() server query in packages/console/app/src/lib/github.ts. Only the stars field is retained, as it is the only field read by any current consumer (footer.tsx, black.tsx, black/workspace.tsx).

The release field's last consumer was removed when the download page was simplified to use a static /releases/latest URL (Dec 2025); the leftover memo in index.tsx was cleaned up in Apr 2026. The contributors field has never had a production consumer since the file was introduced in Oct 2025 — the i18n "{{contributors}}" placeholder is backed by config.stats.contributors, not this value.

AI model, prompt, or tooling used: Detail: Dead Code

How did you verify your code works?

Ran npx tsgo --noEmit (exit 0) and bun test (7 pass, 0 fail) in packages/console/app. Confirmed via grep that no consumer reads .release or .contributors on the github data accessor.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

History


Dead Code PRs can be configured here.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@github-actions

Copy link
Copy Markdown

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • No issue referenced. Please add Closes #<number> linking to the relevant issue.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants