Skip to content

fix(api-docs): bump @absmartly/api-mocks to 1.0.10 - #296

Merged
calthejuggler merged 1 commit into
masterfrom
fix/api-mocks-1.0.10
Oct 7, 2026
Merged

calthejuggler merged 1 commit into
masterfrom
fix/api-mocks-1.0.10

Conversation

@calthejuggler

@calthejuggler calthejuggler commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Fixes the crash on these Web Console API pages:

  • /docs/apis-and-sdks/web-console-api/0100-001-create-experiment/
  • /docs/apis-and-sdks/web-console-api/0100-002-update-experiment/
  • /docs/apis-and-sdks/web-console-api/0100-006-restart-experiment/

Each one showed "This page crashed. n is not iterable". @absmartly/api-mocks@1.0.8 has required: null on the Template schema. The theme loops over required when it merges allOf, so any request body using ExperimentInput crashed. The fix is upstream in absmartly/api-mocks#28, released as 1.0.10.

Verification

  • yarn gen:api no longer prints r is not iterable (3 warnings before, 0 now).
  • yarn build succeeds. All three pages render from the production build with no page errors, and the template_permission fields appear.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated internal development tooling. This update does not change the product’s features, interface, or behavior, and no action is required. There are no new capabilities, fixes, or visible changes to highlight. The update is limited to development and maintenance; the experience of using the product remains unchanged.

1.0.8 shipped a Template schema with `required: null`, which crashed the
create, update and restart experiment Web Console API pages with
"n is not iterable" when the theme merged their allOf request schemas.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: e919f674-1288-46da-85e4-319ffe9239c8
📥 Commits

Reviewing files that changed from the base of the PR and between a1401fb and 2ceea4d.

⛔ Files ignored due to path filters (1)
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (1)
  • package.json

Included review availability: This review used your included allowance. 1 included review remains after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


Walkthrough

The @absmartly/api-mocks development dependency version changes from 1.0.8 to 1.0.10 in package.json.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 2ceea

This bumps the API mocks dev dependency to a version that fixes the documentation page crashes. The version is available from the registry, so merging carries little risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the dependency update to @absmartly/api-mocks version 1.0.10. It matches the main change and is concise.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

I’m a rabbit with a package to inspect,
A newer mock version is now in effect.
From one point-oh-eight to one point-oh-ten,
I hop through the version line again.
Then tuck in my burrow, content and neat.

Comment @coderabbitai help to get the list of available commands.

@netlify

netlify Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for absmartly-docs ready!

Name Link
🔨 Latest commit 2ceea4d
🔍 Latest deploy log https://app.netlify.com/projects/absmartly-docs/deploys/6ac66b116a895e0008021723
😎 Deploy Preview https://deploy-preview-296--absmartly-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@calthejuggler
calthejuggler merged commit 28ff8f8 into master Oct 7, 2026
6 checks passed
@calthejuggler
calthejuggler deleted the fix/api-mocks-1.0.10 branch October 7, 2026 16:01
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.

1 participant