feat(news): add RSSHub presets for CLS, Gelonghui and Jin10 - #1386
Open
2233admin wants to merge 1 commit into
Open
feat(news): add RSSHub presets for CLS, Gelonghui and Jin10#13862233admin wants to merge 1 commit into
2233admin wants to merge 1 commit into
Conversation
|
Someone is attempting to deploy a commit to the luokerenx4's Team Team on Vercel. A member of the Team first needs to authorize it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add opt-in RSSHub presets for 财联社 (CLS telegraph), 格隆汇 (Gelonghui live news), and 金十数据 (Jin10 market news) to Settings → News Sources.
Users supply an RSSHub instance URL reachable from the Alice backend, then add any of the three presets as ordinary RSS subscriptions. OpenAlice does not install or supervise RSSHub, does not depend on a public instance, and does not enable these feeds by default.
Behavior and boundaries
Verification
Executed against this isolated feature branch, based on dev commit 7fdf5ca:
pnpm --package=node@22.23.2 dlx node scripts/run-tests.mjs --changed origin/dev— 4 files, 59 tests passed.pnpm --package=node@22.23.2 dlx node scripts/run-tests.mjs --owner ui --path ui/src/demo/handlers/configNews.spec.ts— 3 tests passed; existing demo handlers round-trip the unchanged feed shape.pnpm --package=node@22.23.2 dlx node node_modules/typescript/bin/tsc -b ui— passed.git diff --check— passed before commit./settings/news-collectorroute indev:demo: invalid URL error, all three preset additions, saved config payload, existing-feed preservation, added-state buttons; desktop and 390px viewport checked, no document horizontal overflow.NewsCollectorandNewsCollectorStore, using temporary isolated storage and a real self-hosted RSSHub. Received CLS 20, Gelonghui 15, Jin10 21 items (56 total). Source-filtered HTTP handlers returned the corresponding stored items. Second collection and collection after storage reload added zero duplicates. Temporary storage removed afterward.diygod/rsshub@sha256:0c36f939df98144fc2cbba1c5d7feefbc1675cc6ad5e99544fa1c77ac84a4e0d.Limits and environment notes
rsshub.appendpoints returned HTTP 403 in the earlier local investigation; this is why no public instance is shipped as a default.cargo test --workspace --no-fail-fastwas attempted from the feature checkout root and could not run because this TypeScript repository has no Cargo.toml. It is not reported as passing.RSSHub implementation references: CLS, Gelonghui, Jin10.
This PR is independent of #1383 and contains none of the unrelated changes in the original local working tree.