Repository navigation
feat(github): add travnie activity source - #471
Conversation
|
This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again. |
 |
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript |  |
Oct 4, 2026 10:44p.m. | Review ↗ |
| Python | |
Oct 4, 2026 10:44p.m. | Review ↗ |
| Shell | |
Oct 4, 2026 10:44p.m. | Review ↗ |
| Secrets | |
Oct 4, 2026 10:44p.m. | Review ↗ |
Important
AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.
![deepsource-io[bot]](https://avatars.githubusercontent.com/in/16372?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c4adda8790
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
cdf5981 to
1b92958
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cdf5981024
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
🐤 Kanarek · 🟣 merged
Feedseek · 3 files |
af47c5f to
bb753e4
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: af47c5f317
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (3)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthrough新增 Travnie GitHub 活动源。抓取器从公开 Events API 请求事件，过滤非公开及无效事件，将有效事件转换为 Feed 条目，并注册到额外抓取器列表。代码还更新来源配额、来源列表和主 Feed 描述，并设置主 Feed 不按标题去重。新增测试覆盖请求、事件转换、来源注册及去重配置。 Sequence Diagram(s)sequenceDiagram
participant GitHubFeed
participant scrape_travnie_activity
participant GitHubEventsAPI
GitHubFeed->>scrape_travnie_activity: 调用已注册的抓取器
scrape_travnie_activity->>GitHubEventsAPI: 请求 Travnie 公开事件
GitHubEventsAPI-->>scrape_travnie_activity: 返回事件列表
scrape_travnie_activity-->>GitHubFeed: 返回规范化条目
Priority: ⬇️ Low Change: Feature Merge Risk: ⚪ Minimal · up to The Travnie source can proceed with normal checks; the supplied evidence does not establish a merge-blocking regression. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to The new source uses public GitHub activity without private-access credentials. No security regression was established, but downstream rendering, link enrichment, and concurrent publication behavior remain incompletely verified. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
Resilience and Maintainability Implications
Hardening Proposals
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 我是小兔，跳到代码旁， Comment |
8b2d30f to
f910dcc
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8b2d30f1f6
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
715c97a to
15028ed
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 715c97a3ae
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f0dff188fb
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 91c2e77791
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| parsed.fragment, | ||
| ) | ||
| ) | ||
| return f"{base_link}#feedseek-event-{quote(event_id, safe='')}" |
There was a problem hiding this comment.
When an event target lacks a real fragment—such as a push commit, pull request, release, or wiki page—this appends a fabricated #feedseek-event-* anchor and publishes it as the Atom/JSON Feed alternate link, so consumers receive a noncanonical URL and browsers attempt to navigate to a nonexistent page anchor. Fresh evidence after the earlier event-identity comments is that the finalized helper stores the ID in link itself; keep base_link as the canonical link and use the event ID as a separate persisted identity and merge key.
AGENTS.md reference: AGENTS.md:L13-L13
Useful? React with 👍 / 👎.
| "DeleteEvent": "deleted a ref from", | ||
| "DiscussionCommentEvent": "commented on a discussion in", | ||
| "ForkEvent": "forked", | ||
| "GollumEvent": "updated wiki pages in", |
There was a problem hiding this comment.
When a GollumEvent contains a page whose action is created, this hardcoded title says the actor merely “updated wiki pages,” silently losing the creation fact. GitHub's event schema explicitly distinguishes created from edited in payload.pages[].action, so derive the title from those page actions instead of assigning every wiki event the same verb.
AGENTS.md reference: AGENTS.md:L14-L14
Useful? React with 👍 / 👎.
1 participant
Adds the requested GitHub organization activity source without committing or using the legacy private Atom token.
travnieorganization events from GitHub's public Events APIpublic: falseevents because Feedseek publishes tracked public feedsThe authenticated private organization dashboard cannot be mirrored 1:1 into a public Feedseek feed without exposing private activity metadata, so this PR intentionally uses the public subset only.
Summary by CodeRabbit