Skip to content

Removing Old Pioneer Entries - #1276

Merged
michaelcanova merged 1 commit into
mainfrom
removing_pioneer_entries
Nov 4, 2026
Merged

michaelcanova merged 1 commit into
mainfrom
removing_pioneer_entries

Conversation

@michaelcanova

@michaelcanova michaelcanova commented Nov 4, 2026 •

Copy link
Copy Markdown
Member

Overview

This PR removes several old pioneer entries, most of which have been converted to proper RR

Summary by CodeRabbit

  • Content Updates
    • Replaced the feed entry about a registered report on psilocybin and ischemic stroke with an entry about research on genes and Flavobacterium covae virulence.
    • Updated the paper and author details, external-metrics search terms, and score for the new entry.

@vercel

vercel Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
web Ready Ready Preview Nov 4, 2026 9:36pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The feed entry for the psilocybin and ischemic stroke report is replaced with an entry for the Flavobacterium covae virulence paper. Paper metadata, external-metrics search terms, and the adjusted score are updated.

Changes: Journal feed entry

Layer / File(s) Summary
Replace paper entry metadata
components/Journal/lib/pioneersFeedEntries.json
The entry’s identifiers, paper and author details, and external search terms now correspond to the Flavobacterium covae virulence paper. The vote and reply counts are 2 and 0, and the adjusted score is 2.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: yattias

Merge Risk: Minimal · up to 5db26

No merge-blocking risk is established for the feed entry replacement.

Architecture risk: Low · up to 5db26

The change affects 1 system.

Changed systems: components

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — components (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in components/Journal/lib/pioneersFeedEntries.json: The feed entry’s paper ID, creation date, hub, subcategory, slug, and unified document ID now identify the Flavobacterium covae virulence paper instead of the psilocybin and ischemic stroke report.
  • observed — Modified behavior in components/Journal/lib/pioneersFeedEntries.json: The replaced paper’s author list, title, abstract, DOI, work type, action date, and publishing author are replaced with details for the Flavobacterium covae virulence paper. The entry’s vote and reply counts are 2 and 0.
  • observed — Modified behavior in components/Journal/lib/pioneersFeedEntries.json: The Bluesky search terms change to the replacement paper’s DOI and title.
  • observed — Modified behavior in components/Journal/lib/pioneersFeedEntries.json: The GitHub mentions search terms change to the replacement paper’s DOI and title, and the entry’s adjusted score changes from 1 to 2.
Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "Removing Old Pioneer Entries" matches the PR objective of removing several old pioneer entries. The raw summary describes a different change: one feed entry about a psilocybin Stage 1 regis…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "Removing Old Pioneer Entries" matches the PR objective of removing several old pioneer entries. The raw summary describes a different change: one feed entry about a psilocybin Stage 1 registered report is replaced with an entry on unique genes and Flavobacterium covae virulence, and the adjusted score changes from 1 to 2. The diff shows 1449 lines removed and only 54 added, which fits removal of many entries, but the summary does not mention the removals. The title is therefore partially related: it names the PR's stated intent, but it does not describe the replacement shown in the summary.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 98e57eef-3953-4ca4-a2ba-81360f7dd0e2
Commits

Reviewing files that changed from the base of the PR and between 1f389c8 and 5db2662.

Files selected for processing: 1
  • components/Journal/lib/pioneersFeedEntries.json

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

@sonarqubecloud

sonarqubecloud Bot commented Nov 4, 2026

Copy link
Copy Markdown

@michaelcanova
michaelcanova marked this pull request as ready for review November 4, 2026 21:39
@michaelcanova
michaelcanova requested a review from a team as a code owner November 4, 2026 21:39
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-11-04T21:39:44.421076Z
We couldn’t complete this review. Comment @codex review to try again.
5db2662 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@michaelcanova
michaelcanova merged commit db42ec7 into main Nov 4, 2026
5 checks passed
@michaelcanova
michaelcanova deleted the removing_pioneer_entries branch November 4, 2026 21:47

This branch was successfully deployed

1 active deployment
Preview — 5db26626 Deployed Nov 4, 2026 by vercel[bot]
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