Skip to content

Resolve map and mapMany profiles through a precomputed index - #32

Merged
stackoverfloweth merged 1 commit into
kitbagjs:mainfrom
asg86260:index-profile-lookup
Sep 30, 2026
Merged

stackoverfloweth merged 1 commit into
kitbagjs:mainfrom
asg86260:index-profile-lookup

Conversation

@asg86260

Copy link
Copy Markdown
Contributor

Description

  • map/mapMany now look up profiles through an index built once per registry instead of re-filtering every registered profile on each call.
  • Type-check cost scales poorly with registry size today; in a ~160-profile app this cut vue-tsc peak memory ~11% and instantiations ~23%.
  • Accepted keys, source types and destination types are unchanged; no new any.

Testing

  • npm run types, npm run lint, and vitest --typecheck pass.
  • Probe files covering wrong keys/sources/destinations, async and union-param profiles produce identical errors before and after.

Development Checklist

  • All code is complete, compiles, and when applicable unit/component tests are written and pass.

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

🤖 Generated with Claude Code

Group registered profiles by source key, then destination key, once per
registry so each call is an indexed lookup instead of re-filtering the
whole profile union. Type behaviour is unchanged.

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

netlify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for kitbag-mapper ready!

Name Link
🔨 Latest commit 812e877
🔍 Latest deploy log https://app.netlify.com/projects/kitbag-mapper/deploys/6abd4dc38680f900087fec8a
😎 Deploy Preview https://deploy-preview-32--kitbag-mapper.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@stackoverfloweth stackoverfloweth left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @asg86260!

@stackoverfloweth
stackoverfloweth merged commit 9834e28 into kitbagjs:main Sep 30, 2026
4 checks passed
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.

2 participants