Repository navigation
Send related essay links to kindel.com/essays - #5
Conversation
related.html rewrites an Essays-category post to the apex essay URL. Other blog links stay put.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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[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: 05c6699a7b
ℹ️ 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".
A missed WordPress page used to keep the posts already fetched. That hides every essay on the pages that did not load. Any failed page now drops that partial list and uses data/essay_slugs.json.
There was a problem hiding this comment.
🟡 Changes recommended
Incomplete pagination can bypass the snapshot fallback, and rewritten links can alter encoded fragment targets.
2 open findings
What changed in this PR
Routes shared related-essay links to kindel.com for Essays-category posts, leaving other blog links unchanged.
Changes:
- Adds URL rewriting backed by live WordPress data and an offline snapshot.
- Mounts and documents the snapshot.
- Adds snapshot and link validation tests.
| File | Description |
|---|---|
| tests/test_essay_links.py | Checks snapshot contents and link conventions. |
| README.md | Documents routing and snapshot mounting. |
| layouts/partials/app-kit/related.html | Applies the URL rewrite helper. |
| layouts/partials/app-kit/essay-slug.html | Validates essay slugs. |
| layouts/partials/app-kit/essay-href.html | Rewrites eligible blog links. |
| layouts/partials/app-kit/essay-catalog.html | Loads live or offline essay mappings. |
| hugo.toml | Mounts module data. |
| data/essay_slugs.json | Supplies offline slug and ID mappings. |
🧠 Review effort: Balanced
Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.
The related-essays line rewrites a post in the WordPress Essays category (slug essays, id 448) to https://kindel.com/essays//. Other blog links stay on blog.kindel.com.
The partial reads the live category when it can, and data/essay_slugs.json when it cannot. The note in that file says how to refresh it. kindelwww carries its own copy of these partials until this module is pinned, because the host sets its own mounts.
Refs kindel/kindelwww#261