Skip to content

feat: real probe latency + city client polish (author edits, as-is) - #22

Merged
arko33 merged 1 commit into
mainfrom
feat/probe-latency-city-polish
Oct 21, 2026
Merged

arko33 merged 1 commit into
mainfrom
feat/probe-latency-city-polish

Conversation

@arko33

@arko33 arko33 commented Oct 21, 2026

Copy link
Copy Markdown
Owner

Author change, committed exactly as written:

  • probes.mjs: p95 chart finally gets real numbers — health probe latency is measured (Date.now delta around the fetch) and recorded as latencyMs on success and on failure (with httpStatus); previously it was always null.
  • city-client.mjs / city.css: further polish on the 3D city client (+180 lines, +31 css).
  • shadowgit.mjs: additional hardening (+/-38 lines).
    node --check green on all touched .mjs files.

@vercel

vercel Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
ghostinthecodebase Ready Ready Preview Oct 21, 2026 11:38pm UTC

@testsprite

testsprite Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Caution

Failed Widespread failures — 23 failed

The executable subset of this backend suite did not produce any product-level assertions: every non-passing case was reported as a TestSprite internal error, and there were no Blocked tests. Because the failures do not include API status codes or response bodies, the score is based only on the runnable subset and reflects limited confidence rather than proven functional defects. The one passing integration test suggests at least some invoice visibility/retrieval behavior is in place, but the broader create/read/auth/validation coverage could not be evaluated from this run. Overall reliability is currently hard to judge because the harness errors prevented the suite from exercising the backend meaningfully.

Results: Passed 1 passed • Failed 23 failed • 24 total  ·  View Full Report ↗


Failed 23 failed tests

Failed Create and read invoice as Alice, ensuring owner isolation fields

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Reject invoice listing with unknown user identity

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Create an invoice then read it back

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Get own seeded invoice by id as Bob

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Create invoice with non-numeric amount returns 422 and creates nothing

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Create invoice with malformed JSON body returns 400 invalid_json

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Reject invoice listing without authentication

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Unauthenticated request to invoices endpoint should not reveal data even if caller is otherwise valid

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed List invoices with a real user session returns only that user's seeded invoice

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Invoice list response shape is stable for seeded invoices

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Create invoice with missing customer returns 422 and creates nothing

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Return 404 for an unknown invoice id

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Get own seeded invoice by id as Alice

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Reject unauthenticated invoice read

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed List invoices does not leak other users' invoice records or fields

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Deny access when Alice reads Bob's invoice

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed List bob's own invoices

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Unknown x-user-id value returns 401 on POST /api/invoices

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Create invoice without x-user-id returns 401 and no invoice is created

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Unknown invoice id returns not_found even for authenticated user

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed List alice's own invoices

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Read a created invoice using the id captured from create

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)


Failed Reject missing auth before invoice existence is evaluated

Screenshot Error details

—

TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)

Passed 1 passed test
Test Case Status Screenshot
Invoice visibility and retrieval flow Passed Passed —
Evaluation Evaluation details

What could be better What could be better: The main issue is test harness execution, not a confirmed backend contract failure: 24 tests failed with the same internal error reference (TestSprite hit an internal error while running this test; our team has been notified. (ref: 0f6d37d9-bdb3-45be-8b48-a0c02e49b184)). That means we do not have evidence yet for whether POST /api/invoices, invoice reads, auth checks, or validation are correct. The suite’s intended coverage is strong—401/403, 422, 404, owner isolation, and list-shape checks—but the run did not surface actual status codes or response bodies. The project needs a clean rerun with a stable harness before any backend behavior can be trusted.

Next steps Recommended next steps: First, rerun the backend suite after fixing the TestSprite execution issue so the tests can reach the API and return real assertions. Once the harness is healthy, focus on the highest-value invoice flows: authenticated create/read, owner isolation on reads and list endpoints, and request validation for missing fields and malformed JSON. Verify that unauthenticated requests are rejected before existence checks, and that unknown IDs return the documented 404/not_found behavior. If the rerun still fails, inspect server logs and test setup for auth fixture or environment mismatches. Use the fix prompt below to hand the agent the exact failing areas to investigate.


Automated end-to-end testing powered by TestSprite

@arko33
arko33 merged commit 77bde49 into main Oct 21, 2026
4 checks passed
@arko33
arko33 deleted the feat/probe-latency-city-polish branch October 21, 2026 23:39
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.

1 participant