Skip to content

Conversation

@lihbr
Copy link
Member

@lihbr lihbr commented Jan 13, 2026

Resolves: https://prismic-team.slack.com/archives/CPG31MDL1/p1767962753033489

Related:

Description

  • Removed statuses CJS dependency
  • Maintained package structure as per 2026 template (notably switching to tsdown for building the package)

Checklist

  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

How to QA 1

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

@lihbr
Copy link
Member Author

lihbr commented Jan 13, 2026

TODO: Consider turning @prismicio/client into optional peer deps

Copy link
Member

@angeloashmore angeloashmore left a comment

Choose a reason for hiding this comment

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

LGTM! I left a question about an old error message, not related to the changes in this PR.

): ErrorResponseMessage<TError> => {
if (status < 400) {
throw new TypeError(
`Invalid error status code, expected status to be within \`[500;600[\`, got \`${status}\``,
Copy link
Member

Choose a reason for hiding this comment

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

Should this error message contain [400, 600), not [500;600[?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice catch!

@angeloashmore
Copy link
Member

@lihbr: TODO: Consider turning @prismicio/client into optional peer deps

Since @prismicio/client's types are used throughout the package, such as SliceZone, it shouldn't be optional. If a project didn't install it, consumers would see any in some types. Making it a peer dependency is probably a good idea though.

lihbr and others added 3 commits January 14, 2026 07:06
Co-authored-by: Angelo Ashmore <angeloashmore@users.noreply.github.com>
@lihbr lihbr merged commit 8a48fd2 into master Jan 14, 2026
11 checks passed
@lihbr lihbr deleted the lh/maintain branch January 14, 2026 01:24
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.

3 participants