Skip to content

fix: CI secret scan and benchmark debug logging after LoCoMo merge - #293

Draft
cursor[bot] wants to merge 1 commit into
masterfrom
cursor/overall-code-stability-a7e1
Draft

fix: CI secret scan and benchmark debug logging after LoCoMo merge#293
cursor[bot] wants to merge 1 commit into
masterfrom
cursor/overall-code-stability-a7e1

Conversation

@cursor

@cursor cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Post-merge stability review of feat: real LoCoMo benchmarks on live Neo4j/Qdrant (no mock) (e9fbaae). Full CI matrix validated locally; restored CI secret-scan exclusions that were missing from master and gated benchmark DEBUG logging behind BENCHMARK_DEBUG=1.

Type

  • feat — new feature
  • fix — bug fix
  • chore — tooling, deps, repo hygiene
  • docs — documentation only
  • ci — GitHub Actions / automation

Testing

go build ./...
go test -short ./...
go vet ./...
# secret scan (with exclusions)
! rg -n "(sk_live_|sk_test_|ghp_|gho_|AKIA|BEGIN (RSA |OPENSSH )?PRIVATE KEY)" \
  --glob '!*.md' --glob '!*.mdx' --glob '!.github/**' --glob '!docs/**' \
  --glob '!internal/memory/privacy/**' --glob '!dashboard/tests/**' .
# frontend
(cd dashboard && npm ci --legacy-peer-deps && npm run build && npm run build:open)
(cd landing && npm ci && npm run build)
(cd sdk/nodejs && npm ci && npm run typecheck && npm run build)
(cd sdk/python && pip install -e '.[dev]' && ruff check . && python3 -m pytest -q)
(cd docs && npx mintlify@latest validate)
  • Built locally
  • Tested affected endpoints/components

Deploy impact

  • Landing (hystersis.com)
  • Docs (docs.hystersis.com / /docs)
  • API (api.hystersis.com)
  • Dashboard (app.hystersis.com)
  • None

Agent notes

LoCoMo commit review: dataset loads (5882 memories / 1542 questions), convert_locomo.py works, evaluation tests pass. Live Neo4j/Qdrant benchmark not run in this environment (no Docker); use ./scripts/product-readiness.sh where stores are available.

Open in Web View Automation 

Re-apply ripgrep install and path exclusions for docs, privacy tests, and
dashboard security fixtures so CI Security job passes on master.

Gate benchmark fmt.Printf DEBUG output behind BENCHMARK_DEBUG=1 to avoid
noisy logs during normal benchmark runs.

Co-authored-by: Himan <Himan-D@users.noreply.github.com>
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
agent-memorydash 3dc155b Jul 17 2026, 06:24 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
hystersis-app 3dc155b Jul 17 2026, 06:28 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
hystersis-docs 3dc155b Jul 17 2026, 06:30 PM

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
agent-memory 3dc155b Jul 17 2026, 06:33 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant