Skip to content

Add Dolphin Cove cartoon dolphin app - #17

Merged
setnilson merged 2 commits into
masterfrom
add-dolphin-cove-20261008-7IgK0w
Oct 8, 2026
Merged

setnilson merged 2 commits into
masterfrom
add-dolphin-cove-20261008-7IgK0w

Conversation

@setnilson

Copy link
Copy Markdown
Owner

Summary

App Builder

The Dolphin Cove session added a cartoon dolphin app with permanent identifier a4fb6afa-195c-48ce-9dde-5229f6ff759f. That app was merged in PR #16. This follow-up repairs the shared lockfile after the post-merge Lockfile consistency job failed and prevented upload.

Changes

  • Regenerate package-lock.json using npm 10.9.2, restoring eight nested dependency records required by existing workspaces: action-catalog 0.0.3, apps-frontend 0.0.5, and zod 4.3.6.
  • Preserve app code, identifiers, manifests, and existing dependency versions. The original Dolphin Cove SVG scene, splash interaction, mobile layout, and reduced-motion support remain intact.
  • Incorporate origin/master at 239a707.

Testing

  • Reproduced npm ci failure with npm 10.9.2 on Node 22; npm 11.6.2 had incorrectly accepted the incomplete lockfile during the initial implementation.
  • After regeneration, npm ci --ignore-scripts and npm ci both pass with Node 22/npm 10.
  • Verified workspace records, links, React ranges, and React 18.3.1 resolution for all 12 workspaces.
  • Full workspace typecheck and lint pass. All workspace builds pass; six initially failed because this sandbox exports an empty Datadog site variable, then passed with those empty variables removed for the build commands.
  • Remote CI has not yet validated this follow-up. A lockfile-only change does not select an app for upload in the current deployment workflow; live deployment remains unverified.
  • Original app browser checks covered splash interaction, animation completion, reduced motion, and mobile overflow; no UI changes in this follow-up.

PR by Bits - View session in Datadog

Comment @datadog-staging to request changes

datadog-bits-staging and others added 2 commits October 8, 2026 21:12
…261008

Co-authored-by: setnilson <110850377+setnilson@users.noreply.github.com>
Co-authored-by: setnilson <110850377+setnilson@users.noreply.github.com>
@datadog-staging

datadog-staging Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

View session in Datadog

Bits Code status: ✅ Done

Comment @datadog-staging to request changes

@setnilson
setnilson merged commit f84161f into master Oct 8, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants