Skip to content

chore: drop orphaned npm manifest, stale docs claim, dead release script#115

Merged
arzafran merged 3 commits into
mainfrom
nr/hy
Jul 9, 2026
Merged

chore: drop orphaned npm manifest, stale docs claim, dead release script#115
arzafran merged 3 commits into
mainfrom
nr/hy

Conversation

@arzafran

@arzafran arzafran commented Jul 9, 2026

Copy link
Copy Markdown
Member

What this does

Removes three things nothing uses.

Summary

  • package.json (vercel) + bun.lock deleted; verified nothing references them (the CLI/daemon hits are opencode's shadow-dir files, not these)
  • CLAUDE.md no longer claims a web/ docs page renders the changelog (no web/ exists here)
  • scripts/build-sign-upload.sh deleted: orphaned duplicate of release.yml's codesign/notarize/DMG logic; its only reference was a substring assertion in tests/test_ci_universal_release_settings.sh, updated

Found while here: that test file is not wired into any CI workflow and was already failing for an unrelated reason (setup.sh lacks the string it checks). Left as-is; may deserve its own issue.

Fixes #103.

Test Plan

  • app build green locally (no Swift changes)
  • CI green

arzafran added 3 commits July 9, 2026 14:15
package.json (vercel dep) and bun.lock are not referenced by any
workflow, script, doc, or test in this repo.

Refs #103.
CLAUDE.md referenced web/app/docs/changelog/page.tsx, which does not
exist in this repo. CHANGELOG.md is the sole changelog source; there
is no docs site rendering it.

Refs #103.
scripts/build-sign-upload.sh duplicated release.yml's codesign/
notarize/DMG logic but was never invoked by any workflow or script —
release.yml has its own independent implementation of the same steps.
The only remaining reference was a substring-grep in
tests/test_ci_universal_release_settings.sh checking for the
-Dxcframework-target=universal flag; drop it from that check along
with the script.

Note: the same test independently fails on scripts/setup.sh missing
that flag — pre-existing, unrelated to this cluster, left as-is.

Refs #103.
@arzafran arzafran merged commit 7b383f6 into main Jul 9, 2026
8 checks passed
@arzafran arzafran deleted the nr/hy branch July 9, 2026 22:00
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.

Delete orphaned npm manifest, stale CLAUDE.md web/ reference, and verify build-sign-upload.sh

1 participant