Commit graph

4 commits

Author SHA1 Message Date
zwxxb
282d82f1e0 refactor(move): simplify & harden the Move integration architecture (reduced scope)
Trimmed rebuild of 6ea5aa44 (fork PR #4): Move core (gitnexus/src/core/move/)
and Move tests, plus the functional minimum elsewhere.

Dropped from the original commit:
- .gitignore .codex/ entry (unrelated)
- cli/analyze.ts env-constant swap (cosmetic; same string literal)
- core/logger.ts warnRespectingProgressBar extraction and the
  filesystem-walker.ts reuse of it (Move code never imports the helper)
- scope-resolution callable-value-flow warning aggregation and pipeline/run.ts
  progress-warning formatting, with their two tests (warning-UX hardening,
  not Move-functional)
- process-detection interface doc-comment retuning (DEFAULT_CONFIG runtime
  values already match upstream; region left byte-identical to upstream)

Kept outside src/core/move/ and Move tests (one line each):
- gitnexus-shared/src/graph/types.ts: 'external' locationFidelity for
  dependency symbols
- gitnexus-shared/src/lbug/schema-constants.ts: 'Type' node table registration
- core/lbug/schema.ts: Type table schema + Move rel-table endpoint pairs
- core/lbug/node-table-layout.ts: TYPE_LAYOUT CSV layout
- core/lbug/csv-generator.ts: 'Type' in MULTI_LANG_TYPES routing
- core/lbug/lbug-adapter.ts: deleteAllExternalNodes + shared delete-by-label
  mechanics; Type backtick entry
- core/incremental/subgraph-extract.ts: external nodes get the
  delete-all-then-rebuild treatment
- core/run-analyze.ts: Move consistency digest to meta.json;
  deleteAllExternalNodes call; grouped Move meta; finally-based shutdown
- storage/repo-manager.ts: moveConsistency in RepoMeta;
  INCREMENTAL_SCHEMA_VERSION 13 (Type/EnumVariant persistence)
- core/ingestion/pipeline.ts: generic TStandaloneIngest output threading
- types/pipeline.ts: PipelineResult.standaloneIngest replaces ingestWarnings
- core/ingestion/process-processor.ts: ENTRY_POINT_OF explicit graph roots
- tests: schema, node-table-layout, process-processor,
  repo-manager-reconcile, call-summary-schema-version,
  incremental-subgraph-extract, run-analyze-fts-repair (unit);
  lbug-core-adapter (integration)

Gates: tsc clean; Move unit 185/185; full unit suite green except three
pre-existing Darwin-environment failures reproduced on the pristine base;
integration (move + lbug-core-adapter) 29/29.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-24 18:58:03 +02:00
abhigyantrumio
d61c98a6f9 Merge main-aptos (main sync) and harden cached-install read
- Reconcile the node-table-layout refactor with the main sync that landed
  on main-aptos: springConfig phase registered alongside standaloneIngest,
  Class rows emit the frameworkAnnotations column through the explicit
  code-element path (layout tables unchanged), Move layouts stay the
  single source of truth for DDL/CSV/COPY.
- validCachedInstall now stats and reads metadata through one open file
  handle (symlinks still rejected via lstat first), closing the
  check-then-read race CodeQL flagged (js/file-system-race, alert #918).
- Keep a vendor/move-flow ignore guard (legacy pre-#2622 postinstall
  artifact path; never committed) and drop the vestigial pkg.files
  assertion left from the deleted installer.
- Add an escapeCSVBoolean parity test pinning bulk CSV coercion to the
  incremental path's !!v semantics for non-boolean inputs.
- eval: track the -aptos runtime version in the sandbox containment pin
  (PINNED_GITNEXUS_VERSION must move in lock-step with
  gitnexus/package.json on this branch).

Verified: the actions/cache pin 55cc8345 is genuinely v6.1.0 (the
pre-existing '# v5' label was the mislabeled one), so the new comments
are correct as written.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
2026-07-23 03:15:16 +05:30
zwxxb
cec1a74cf0 fix(move): harden provisioning failure handling
Avoid repeated install attempts and prove compiler-backed indexes remain intact when move-flow disappears. Align docs and CI metadata with managed runtime behavior.

Co-authored-by: Cursor <cursoragent@cursor.com>
2026-07-21 23:58:33 +02:00
zwxxb
30093b2f59 fix(move): provision move-flow on demand at analyze time
Replace the npm-postinstall installer (scripts/install-move-flow.cjs)
with analyze-time provisioning: resolve $MOVE_FLOW, then a verified
user cache under ~/.gitnexus/tools/move-flow, then $PATH, then a
one-time download of the pinned release verified against SHA256SUMS.
Installs are serialized across processes via a lease/heartbeat lock
and staged into place atomically.

- Gate the client probe on the pinned release major (prerelease
  suffixes accepted); drop the legacy vendor/move-flow bundled path.
- Honor GITNEXUS_SKIP_MOVE_FLOW and GITNEXUS_SKIP_OPTIONAL_GRAMMARS;
  do not retry a failed install within the same process.
- Consolidate node-table DDL, CSV headers, row encoding, and COPY
  column lists into lbug/node-table-layout.ts (replaces
  move-columns.ts); align boolean CSV encoding with the incremental
  CREATE path.
- CI: cache ~/.gitnexus/tools/move-flow keyed to the pinned version;
  require provisioning on the shard that owns the live Move suite.
- Docker: install unzip for on-demand extraction; the image no longer
  ships a move-flow binary.
2026-07-21 17:14:08 +02:00