feat: add Content Note bulk upload to simply-core and simply-data-core

[ClayChipps]

Summary

Implements docs/design/0034, converting the scraped UploadContentNotes.md script into library code:

• simply-core: mapConcurrent() — a streaming worker-pool primitive that bounds concurrency over an AsyncIterable/Iterable source, starting the next item as soon as a slot frees rather than requiring the whole input in memory or waiting for a whole chunk (unlike the existing mapChunked).
• simply-data-core: createContentNote() / uploadContentNotes() — create a ContentNote and link it to a record via ContentDocumentLink, with uploadContentNotes driving mapConcurrent over a stream of inputs resolved against a caller-supplied external-id lookup Map.

Fixes several bugs found in the original script along the way:

• CreatedDate was sent as a numeric epoch instead of an ISO-8601 string.
• An undefined-variable reference in the ContentDocumentLink failure branch (errorWriteStreamWriterStream) would crash the run.
• That branch also mislabeled which create call actually failed and dropped the already-created ContentNote's id, losing track of an orphaned note. createContentNote now reports stage: 'link' with contentNoteId still populated.
• Debug console.log of every note's base64 body is gone.

Drops the script's CLI-only concerns (arg parsing, spinner, Org.create(), --output-dir/--use-cached file management) — the library takes an already-resolved Connection and streams typed results back, matching every other -core package here. A CLI command, if wanted, is left as a follow-up per the design doc's open questions.

Also updates docs/design/0034-content-note-upload.md's Status line and corrects two details the implementation clarified: mapConcurrent's source type is AsyncIterable<T> | Iterable<T> (broader than originally drafted), and it propagates a mapper rejection immediately (matching mapChunked's existing convention) rather than the self-contradictory "catches it and yields it" wording in the original draft.

Note: pnpm-lock.yaml's diff is large but is a formatting-only normalization to the pinned pnpm@11.21.0's canonical output, produced by pnpm add @simplysf/simply-core --filter @simplysf/simply-data-core (per CONTRIBUTING.md) — not a dependency version change beyond that one new workspace link.

Test plan

• pnpm run compile / lint / test:only pass for both simply-core and simply-data-core.
• lerna run build/test --scope=@simplysf/simply-core --scope=@simplysf/simply-data-core pass.
• Pre-push hook additionally ran tests for simply-core's dependents (simply-apex-core, simply-community-core, simply-package-core, simply-permissions-core) — all pass, confirming the new export doesn't affect them.
• test/index.test.ts export-pinning updated in simply-data-core.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ASL1LuYtGCPVDcqtqaNvLy