Skip to content

feat/rss feed#738

Open
lucrousseau21 wants to merge 2 commits into
mainfrom
feat/rss-feed
Open

feat/rss feed#738
lucrousseau21 wants to merge 2 commits into
mainfrom
feat/rss-feed

Conversation

@lucrousseau21
Copy link
Copy Markdown
Member

Closes #676

@lucrousseau21 lucrousseau21 marked this pull request as ready for review May 11, 2026 10:12
@lacatoire
Copy link
Copy Markdown
Member

Bonne initiative de proposer un feed pour les agrégateurs. Points à reprendre :

  • Parser Markdown maison cassé sur les fences : src/app/feed.xml/route.ts:34 n'a aucune gestion des triples backticks. Tous les articles techniques finissent rendus en <p> plat avec délimiteurs visibles et indentation perdue. À remplacer par remark/marked.
  • toIsoDate() non déterministe : :73, :76, :96 retombent sur new Date().toISOString() en fallback → timestamps différents à chaque build, les agrégateurs marquent les entrées comme modifiées sans raison. Fallback fixe nécessaire.
  • Regex liste fragile : [-*]\s+ (:54) matche aussi les emphases markdown commençant par * (un *italique* ... est promu en <li>).
  • Hors scope : imageCaption/imageGeoLocation ajoutés à BlogPost mais jamais consommés (code mort), eslint-disable ajouté à ResponsiveImage.tsx:31 sans rapport avec le feed. À sortir dans un PR dédié.
  • Titre PR "RSS" alors que c'est Atom 1.0 : pas grave, mais clarifier dans la description.

Un test couvrant un article avec fence ```ts typique du blog sécuriserait le rendu réel.

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.

SEO: flux RSS/Atom optimisé pour Feedly, NewsBlur, Google News

2 participants