Skip to content

chore(quality): zero-warning lint sweep + synapse memory stability [Story 123.7]#639

Open
rafaelscosta wants to merge 1 commit intorelease/5.0.8-installer-recoveryfrom
feat/story-123.7-lint-zero-and-synapse-stability
Open

chore(quality): zero-warning lint sweep + synapse memory stability [Story 123.7]#639
rafaelscosta wants to merge 1 commit intorelease/5.0.8-installer-recoveryfrom
feat/story-123.7-lint-zero-and-synapse-stability

Conversation

@rafaelscosta
Copy link
Copy Markdown
Collaborator

Summary

Story 123.7 — Zero-warning lint sweep + Synapse Memory Provider stability.

  • Lint sweep across helpers, doctor checks, graph-dashboard, ids, synapse layers, data scripts, installer, scripts, and tests: trailing commas, dead-var/import removal, formatting alignment.
  • tests/synapse/synapse-memory-provider.test.js: refactor mock to use injectable _getLoader on provider instance (fixes intermittent failure caused by jest.doMock + isolateModules ordering).
  • tests/synapse/e2e/pipeline-audit.e2e.js: trailing-comma cleanup.
  • .claude/hooks/precompact-session-digest.cjs: single-quote fix in inlineScript (resolves last 2 lint warnings).
  • New: .synapse/.gitignore — ignore runtime sessions/cache.
  • .gitignore: ignore .agents/ (duplicate of .claude/skills/).

Acceptance criteria verified

  • AC1 — npm run lint → 0 errors, 0 warnings
  • AC2 — npx jest tests/synapse/synapse-memory-provider.test.js → 22/22 pass
  • AC3 — npm run typecheck → clean
  • AC4 — full npm test → deferred to CI

Notes

This is a stacked PR. Base = release/5.0.8-installer-recovery (PR #635). Once #635 merges to main, retarget this PR to main.

🤖 Generated with Claude Code

…tory 123.7]

- Lint sweep across helpers, doctor checks, graph-dashboard, ids,
  synapse layers, data scripts, installer, scripts, and tests:
  trailing commas, dead-var/import removal, formatting alignment.
- tests/synapse/synapse-memory-provider.test.js: refactor mock to use
  injectable _getLoader on provider instance (fixes intermittent failure
  caused by jest.doMock + isolateModules ordering).
- tests/synapse/e2e/pipeline-audit.e2e.js: trailing-comma cleanup.
- .claude/hooks/precompact-session-digest.cjs: single-quote fix in
  inlineScript (resolves last 2 lint warnings).
- New: .synapse/.gitignore — ignore runtime sessions/cache.
- .gitignore: ignore .agents/ (duplicate of .claude/skills/).

Verified:
- npm run lint → 0 errors, 0 warnings
- npx jest tests/synapse/synapse-memory-provider.test.js → 22/22 pass
- npm run typecheck → clean

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
aiox-core Ready Ready Preview, Comment Apr 28, 2026 2:32pm

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 28, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: b983a2d5-1709-4cfc-bcfd-34f31bced777

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/story-123.7-lint-zero-and-synapse-stability

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added area: agents Agent system related area: workflows Workflow system related squad mcp type: test Test coverage and quality area: core Core framework (.aios-core/core/) area: installer Installer and setup (packages/installer/) area: synapse SYNAPSE context engine area: cli CLI tools (bin/, packages/aios-pro-cli/) area: pro Pro features (pro/) area: health-check Health check system area: docs Documentation (docs/) area: devops CI/CD, GitHub Actions (.github/) needs-po-review labels Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: agents Agent system related area: cli CLI tools (bin/, packages/aios-pro-cli/) area: core Core framework (.aios-core/core/) area: devops CI/CD, GitHub Actions (.github/) area: docs Documentation (docs/) area: health-check Health check system area: installer Installer and setup (packages/installer/) area: pro Pro features (pro/) area: synapse SYNAPSE context engine area: workflows Workflow system related mcp needs-po-review squad type: test Test coverage and quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant