Skip to content

docs: add gitnexus-shared build step before gitnexus-web#585

Merged
magyargergo merged 1 commit into
abhigyanpatwari:mainfrom
agungwa:docs/add-gitnexus-shared-build-step
Mar 30, 2026
Merged

docs: add gitnexus-shared build step before gitnexus-web#585
magyargergo merged 1 commit into
abhigyanpatwari:mainfrom
agungwa:docs/add-gitnexus-shared-build-step

Conversation

@agungwa

@agungwa agungwa commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

gitnexus-web imports from gitnexus-shared, which requires npm run build to generate dist/. Without this step, npm run dev fails with module resolution errors.

Summary

Adds the missing gitnexus-shared build step to the web UI quick start instructions in README.md.

Motivation / context

Users following the existing docs get this error when running npm run dev in gitnexus-web:

src/core/llm/tools.ts:16:39 - error: Module '"gitnexus-shared"' has no exported member 'NODE_TABLES'

The gitnexus-shared package has its main entry pointing to dist/index.js, but dist/ doesn't exist until npm run build is run.

Areas touched

  • gitnexus/ (CLI / core / MCP server)
  • gitnexus-web/ (Vite / React UI)
  • .github/ (workflows, actions)
  • eval/ or other tooling
  • Docs / agent config only (AGENTS.md, CLAUDE.md, .cursor/, llms.txt, etc.)

Scope & constraints

In scope

  • Update README.md web UI quick start section to include gitnexus-shared build step

Explicitly out of scope / not done here

  • CONTRIBUTING.md (development setup section) — can be updated separately if needed

Implementation notes

Changed the quick start commands from:

  cd gitnexus/gitnexus-web
  npm install
  npm run dev

To:

  cd gitnexus/gitnexus-shared && npm install && npm run build
  cd ../gitnexus-web && npm install
  npm run dev

Testing & verification

  • cd gitnexus && npm test
  • cd gitnexus && npm run test:integration
  • cd gitnexus && npx tsc --noEmit
  • cd gitnexus-web && npm test
  • cd gitnexus-web && npx tsc -b --noEmit
  • Manual verification — followed updated steps, npm run dev starts successfully

Risk & rollout

No code changes — documentation only. Zero risk.

Checklist

  • PR body meets repo minimum length (workflow may label short descriptions)
  • If AGENTS.md / overlays changed: headers, scope block, and changelog updated per project conventions
  • No secrets, tokens, or machine-specific paths committed

gitnexus-web imports from gitnexus-shared, which requires npm run build
to generate dist/. Without this step, npm run dev fails with module
resolution errors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Mar 29, 2026

Copy link
Copy Markdown

@agungwa is attempting to deploy a commit to the NexusCore Team on Vercel.

A member of the Team first needs to authorize it.

@magyargergo magyargergo merged commit be210c7 into abhigyanpatwari:main Mar 30, 2026
1 of 2 checks passed
icodebuster pushed a commit to icodebuster/GitNexus that referenced this pull request Mar 31, 2026
* main: (114 commits)
  feat(csharp): C# MethodExtractor config (abhigyanpatwari#582)
  docs: add gitnexus-shared build step before gitnexus-web (abhigyanpatwari#585)
  chore: add enterprise offering section to README, ignore local_docs/ (abhigyanpatwari#579)
  fix(eval): exclude litellm 1.82.7 and 1.82.8 due to compatibility issues (abhigyanpatwari#580)
  feat(java,kotlin): MethodExtractor abstraction with per-language configs (abhigyanpatwari#576)
  feat: added skip-agents-md cli flag (abhigyanpatwari#517)
  feat(wiki): Azure OpenAI support for wiki command (abhigyanpatwari#562)
  refactor: reduce explicit any types (abhigyanpatwari#566)
  feat(java): method references, worker overload disambiguation, interface dispatch (abhigyanpatwari#540)
  feat: configure eslint with unused import removal (abhigyanpatwari#564)
  feat: configure prettier with pre-commit hook (abhigyanpatwari#563)
  feat: unify web and cli ingestion pipeline (abhigyanpatwari#536)
  fix/opencode mcp gitnexus timeout (abhigyanpatwari#363)
  chore: bump version to 1.4.10, update CHANGELOG
  fix: resolve tree-sitter peer dependency conflicts (abhigyanpatwari#538)
  chore: bump version to 1.4.9, add CHANGELOG.md
  refactor: Phase 8 & 9 — Field Types and Return-Type Binding (abhigyanpatwari#494)
  feat: add COBOL language support with regex extraction pipeline (abhigyanpatwari#498)
  fix: close remaining Dart language support gaps (abhigyanpatwari#524)
  refactor: split global BUILT_IN_NAMES into per-language provider fields (abhigyanpatwari#523)
  ...

# Conflicts:
#	gitnexus/src/core/wiki/llm-client.ts
@abhigyanpatwari abhigyanpatwari mentioned this pull request Apr 1, 2026
3 tasks
motolese pushed a commit to motolese/datamoto-gitnexus that referenced this pull request Apr 23, 2026
…wari#585)

gitnexus-web imports from gitnexus-shared, which requires npm run build
to generate dist/. Without this step, npm run dev fails with module
resolution errors.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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