Skip to content

perf(api): cache hot author and find-by-book lookups - #953

Merged
jakeaturner merged 1 commit into
stagingfrom
perf/author-and-book-cache
Aug 30, 2026
Merged

perf(api): cache hot author and find-by-book lookups#953
jakeaturner merged 1 commit into
stagingfrom
perf/author-and-book-cache

Conversation

@jakeaturner

@jakeaturner jakeaturner commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings August 30, 2026 04:08
Copilot AI reviewed Aug 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Aug 30, 2026 •
edited
Loading

Copy link
Copy Markdown

Severity: Medium — server/api/authors.ts:125 and server/api/projects.js:664 stack a 180-second HTTP cache atop the 180-second server cache. A response fetched just before server-cache expiry can remain cached downstream for another 180 seconds, making updates or newly linked records stale for nearly six minutes. Base HTTP freshness on remaining server TTL, or use only one cache layer.

@jakeaturner
jakeaturner force-pushed the perf/author-and-book-cache branch from 4c2f880 to 3007ef6 Compare August 30, 2026 04:14
Copilot AI review requested due to automatic review settings August 30, 2026 04:14
Copilot AI reviewed Aug 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

aikido-pr-checks[bot]
aikido-pr-checks Bot reviewed Aug 30, 2026
View reviewed changes
Comment thread server/api/projects.js
Copilot AI review requested due to automatic review settings August 30, 2026 04:19
@jakeaturner
jakeaturner force-pushed the perf/author-and-book-cache branch from 3007ef6 to 4072d72 Compare August 30, 2026 04:19
Copilot AI reviewed Aug 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@jakeaturner
jakeaturner merged commit 7b5a537 into staging Aug 30, 2026
7 checks passed
@jakeaturner
jakeaturner deleted the perf/author-and-book-cache branch August 30, 2026 04:21
@libretexts-bot

libretexts-bot commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 2.151.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort, defaults to Lite
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
+1 more reviewer
@aikido-pr-checks aikido-pr-checks[bot] aikido-pr-checks[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jakeaturner @libretexts-bot