Skip to content

Search command: fix some critical bugs#11

Merged
ranyitz merged 2 commits into
teambit:masterfrom
davidfirst:search-command
Feb 1, 2017
Merged

Search command: fix some critical bugs#11
ranyitz merged 2 commits into
teambit:masterfrom
davidfirst:search-command

Conversation

@davidfirst
Copy link
Copy Markdown
Member

No description provided.

@ranyitz ranyitz merged commit 29e0f75 into teambit:master Feb 1, 2017
davidfirst added a commit that referenced this pull request Apr 27, 2020
* add public-api links on the box and the root level

* change terminology from box to namespace

* change terminology from box to namespace

* Update CHANGELOG.md
davidfirst added a commit that referenced this pull request May 13, 2026
Audit 06 enumerates every explicit cache-invalidation call site, classifies
each by what it currently invalidates vs. what it needs to invalidate, and
identifies three patterns of over-invalidation:

- Pattern A (#10, #11): callers already have the affected ID list in scope,
  but call clearAllComponentsCache anyway. Mechanical swap to
  clearComponentsCache(ids). Tasks 8.11.
- Pattern B (#12, #4-6, #13): callers know the seed IDs but not the full
  affected set (e.g. auto-tag dependents, env consumers). Each needs a
  small reverse-index. Tasks 8.12, 8.14.
- Pattern C (#3, #8, #9): callers genuinely don't know the affected set.
  Each handled case-by-case: dead-code check, file-read narrowing, or
  accept worst-case nuke. Tasks 8.13, 8.15.

Audit also documents 5 "OK, don't regress" per-id sites and 3 FS-cache
sites, plus the two indirect cascade triggers (bitmap invalidate, configStore
invalidate) which are already minimal.

15 full-clear sites total, 7 of them are over-clears. The largest expected
wins are from narrowing snapping's pre-tag/snap clear (8.12) and the
install-time clears (8.14, deferred until Lever 1 ships and we measure).

No code changes — audit + task planning only.
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