Skip to content

[#38] api communications costs summary - #38

Open
junie-the-developer[bot] wants to merge 1 commit into
mainfrom
yana/api-communications-costs-summary
Open

[#38] api communications costs summary#38
junie-the-developer[bot] wants to merge 1 commit into
mainfrom
yana/api-communications-costs-summary

Conversation

@junie-the-developer

@junie-the-developer junie-the-developer Bot commented Aug 25, 2026

Copy link
Copy Markdown

What

Adds API-server endpoints for communications data and cost summaries so clients can retrieve conversation activity and usage-cost information through a focused HTTP interface. The implementation is accompanied by targeted gateway tests covering endpoint behavior and response handling.

Changes

  • Step 1: Added communications and cost summary API endpoints in the API-server platform and covered them with focused gateway tests.

@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown

૮ >ﻌ< ა ci review

ran on db1dce1 — feat(api_server): add communications and cost summary endpoi

⚠️ Warnings

OSV vulnerability scan · View job

5 known vulnerabilities found in pinned dependencies.

How to fix:

Review the findings in the Security tab. Update the affected dependencies if a patched version is available.


debug info

CI timings

CI timings · View report · View job

Wall time 3m15s vs 7m8s (-54.4%). 13 job(s) slower, 8 faster, 3 unchanged.

  • Python tests / Run tests slice 4/12: +43.0s
  • Python tests / Run tests slice 11/12: +33.0s
  • Python tests / Run tests slice 10/12: +30.0s
  • Python tests / Run tests slice 7/12: -23.0s
  • Python tests / Run tests slice 5/12: +14.0s

@junie-the-developer

Copy link
Copy Markdown
Author

Re: CI attribution failure (comment 5406954519 / check-attribution on a6b588e):

Fixed by adding contributors/emails/junie-live-agent@users.noreply.github.comjunielive via scripts/add_contributor.py (no AUTHOR_MAP / release.py edits).

Pushed on yana/api-communications-costs-summary as f0484f407 (chore: map contributor emails). Local python3 scripts/audit_pr_attribution.py passes. Fresh CI should pick up the new head.

Add authenticated GET /api/communications (sanitized gateway session rows
for People) and GET /api/costs/summary (best-effort daily/last-day/last-week
spend by session started_at day). Leave the existing /api/sessions client
projection unchanged and advertise the new paths on /v1/capabilities.

Co-authored-by: Junie <junie@jetbrains.com>
@junie-the-developer
junie-the-developer Bot force-pushed the yana/api-communications-costs-summary branch from f0484f4 to db1dce1 Compare August 25, 2026 08:09
@junie-the-developer junie-the-developer Bot changed the title feat(api_server): communications + cost summary for web-console People/Costs [#38] api communications costs summary Aug 25, 2026
@junie-the-developer

Copy link
Copy Markdown
Author

Re: CI bot comment on db1dce131 — post-rebase status confirmed green (All required checks pass + check-attribution success; PR mergeable=true). OSV dependency warnings in that bot note are pre-existing pinned deps outside this PR's communications/cost-summary API scope and are not treated as blockers for this change set.

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.

0 participants