Skip to content

feat: share publish telemetry across services - #92

Merged
POWERFULMOVES merged 2 commits into
mainfrom
codex/extend-publishermetrics-and-add-telemetry
Sep 29, 2025
Merged

POWERFULMOVES merged 2 commits into
mainfrom
codex/extend-publishermetrics-and-add-telemetry

Conversation

@POWERFULMOVES

Copy link
Copy Markdown
Owner

Summary

  • factor shared publish telemetry helpers into services/common/telemetry.py
  • extend the publisher and Discord publisher services to expose turnaround/latency metrics and persist Supabase rollups
  • document the telemetry endpoints and ROI dashboard guidance in the roadmap and supporting docs

Testing

  • pytest pmoves/services/publisher/tests/test_publisher.py

https://chatgpt.com/codex/tasks/task_b_68da91415cb883249c4ea74255fa1a3d

@coderabbitai

coderabbitai Bot commented Sep 29, 2025

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@POWERFULMOVES has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 12 minutes and 0 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 5ca96b1 and 1e3c723.

📒 Files selected for processing (7)
  • pmoves/docs/NEXT_STEPS.md (2 hunks)
  • pmoves/docs/ROADMAP.md (2 hunks)
  • pmoves/docs/TELEMETRY_ROI.md (1 hunks)
  • pmoves/services/common/telemetry.py (1 hunks)
  • pmoves/services/publisher-discord/main.py (7 hunks)
  • pmoves/services/publisher/README.md (1 hunks)
  • pmoves/services/publisher/publisher.py (3 hunks)

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

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

@POWERFULMOVES
POWERFULMOVES merged commit d6c8df1 into main Sep 29, 2025
0 of 2 checks passed
POWERFULMOVES pushed a commit that referenced this pull request Dec 25, 2025
…4762

Upgrade python-multipart from 0.0.6 to 0.0.18 in agentgym-rl-coordinator
to address:
- CVE-2024-53981 (High) - DoS via crafted form data
- CVE-2024-24762 (High) - DoS via crafted form data

Fixes Dependabot alerts #91 and #92

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
POWERFULMOVES added a commit that referenced this pull request Jan 18, 2026
…cs-and-add-telemetry

feat: share publish telemetry across services
POWERFULMOVES pushed a commit that referenced this pull request Jan 18, 2026
…4762

Upgrade python-multipart from 0.0.6 to 0.0.18 in agentgym-rl-coordinator
to address:
- CVE-2024-53981 (High) - DoS via crafted form data
- CVE-2024-24762 (High) - DoS via crafted form data

Fixes Dependabot alerts #91 and #92

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
POWERFULMOVES pushed a commit that referenced this pull request Feb 28, 2026
…02-28

Update all three audit tracking documents with live API-verified data:

- Dashboard: 43 CodeQL alerts (35 error, 8 warning), 7 Dependabot (5H, 2L),
  1 open PR (#717). Added Recent Merge Activity for 7 PRs (#712-719).
  CodeQL Groups A-E marked FIXED (PR #715), Group H added for new alerts.
- Hardening tracker: Dependabot High marked REGRESSION (5 new alerts),
  CodeQL footnoted for expanded scope. Added Recent Activity section.
- Merge tracker: PRs #577-581 all CLOSED, Archon #6 CLOSED, DoX #92 MERGED.
  Updated Quick Status with live counts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@POWERFULMOVES
POWERFULMOVES deleted the codex/extend-publishermetrics-and-add-telemetry branch March 7, 2026 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant