refactor: rename trogon-sink-github to trogon-source-github by yordis · Pull Request #91 · TrogonStack/trogonai

yordis · 2026-04-02T23:57:13Z

Summary

Rename trogon-sink-github to trogon-source-github to fix the naming — it receives external webhooks and publishes into NATS, which is a source, not a sink
Update all references: Cargo.toml, Dockerfile, service name enum, docker-compose, and code imports

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

cursor · 2026-04-02T23:57:18Z

PR Summary

Low Risk
Low risk rename-only refactor, but it changes binary/package/service identifiers in Docker, telemetry, and runtime entrypoints, which could break deployments if any external references weren’t updated.

Overview
Renames the GitHub webhook receiver crate/binary from trogon-sink-github to trogon-source-github and updates all corresponding references (imports, service name enum, and docs).

Updates containerization/dev tooling to match the new name: docker-compose service/SMEE target, Dockerfile build/entrypoint targets, and Cargo.lock package name entries (plus minor dependency lockfile bumps).

^{Written by Cursor Bugbot for commit 88c673d. This will update automatically on new commits. Configure here.}

coderabbitai · 2026-04-02T23:57:28Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ecb264b3-a354-4672-b89a-c27d80c54002

📥 Commits

Reviewing files that changed from the base of the PR and between dabe16e and 88c673d.

⛔ Files ignored due to path filters (1)

rsworkspace/Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (10)

devops/docker/compose/compose.yml
rsworkspace/crates/acp-telemetry/src/service_name.rs
rsworkspace/crates/trogon-source-github/Cargo.toml
rsworkspace/crates/trogon-source-github/Dockerfile
rsworkspace/crates/trogon-source-github/src/config.rs
rsworkspace/crates/trogon-source-github/src/constants.rs
rsworkspace/crates/trogon-source-github/src/lib.rs
rsworkspace/crates/trogon-source-github/src/main.rs
rsworkspace/crates/trogon-source-github/src/server.rs
rsworkspace/crates/trogon-source-github/src/signature.rs

Walkthrough

This PR renames the trogon-sink-github service to trogon-source-github throughout the codebase. The rename is applied consistently to Docker Compose configuration, Cargo manifests, Dockerfiles, telemetry service enums, and source code references.

Changes

Cohort / File(s)	Summary
Docker & Deployment Configuration `devops/docker/compose/compose.yml`	Service `trogon-sink-github` renamed to `trogon-source-github`; Dockerfile path, webhook URL, and service health dependency updated to reference the new service name.
Telemetry Service Registration `rsworkspace/crates/acp-telemetry/src/service_name.rs`	`ServiceName` enum variant renamed from `TrogonSinkGithub` to `TrogonSourceGithub`; `as_str()` mapping updated to return `"trogon-source-github"` instead of `"trogon-sink-github"`.
Crate Manifest & Build `rsworkspace/crates/trogon-source-github/Cargo.toml`, `rsworkspace/crates/trogon-source-github/Dockerfile`	Package name and binary target in Cargo.toml updated from `trogon-sink-github` to `trogon-source-github`; Dockerfile build references updated to build and package the renamed crate.
Source Code `rsworkspace/crates/trogon-source-github/src/lib.rs`, `rsworkspace/crates/trogon-source-github/src/main.rs`	Crate-level documentation header updated; module imports and telemetry service registration changed to use `trogon_source_github` and `ServiceName::TrogonSourceGithub`.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A service renamed with consistent care,
From sink to source, the changes flow fair,
Docker, configs, and code aligned,
A rename complete, nothing left behind! 🌟

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarizes the main change: renaming trogon-sink-github to trogon-source-github, which is the primary focus of the entire changeset.
Description check	✅ Passed	The description is directly related to the changeset, explaining the rationale for the rename (correcting naming to reflect that it is a source not a sink) and identifying the files affected.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch yordis/rename-sink-to-source

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 · 2026-04-02T23:59:26Z

Code Coverage Summary

Details

Filename                                                                      Stmts    Miss  Cover    Missing
--------------------------------------------------------------------------  -------  ------  -------  ---------------------------------------------------------------------------------------------
crates/acp-nats/src/session_id.rs                                                72       0  100.00%
crates/acp-nats/src/ext_method_name.rs                                           70       0  100.00%
crates/acp-nats/src/req_id.rs                                                    39       0  100.00%
crates/acp-nats/src/jsonrpc.rs                                                    6       0  100.00%
crates/acp-nats/src/lib.rs                                                       73       0  100.00%
crates/acp-nats/src/client_proxy.rs                                             200       0  100.00%
crates/acp-nats/src/config.rs                                                   204       0  100.00%
crates/acp-nats/src/acp_prefix.rs                                                51       0  100.00%
crates/acp-nats/src/error.rs                                                     84       0  100.00%
crates/acp-nats/src/pending_prompt_waiters.rs                                   141       0  100.00%
crates/acp-nats/src/in_flight_slot_guard.rs                                      32       0  100.00%
crates/trogon-std/src/env/in_memory.rs                                           81       0  100.00%
crates/trogon-std/src/env/system.rs                                              17       0  100.00%
crates/acp-nats/src/nats/subjects/responses/ext_ready.rs                         15       0  100.00%
crates/acp-nats/src/nats/subjects/responses/response.rs                          20       0  100.00%
crates/acp-nats/src/nats/subjects/responses/update.rs                            27       0  100.00%
crates/acp-nats/src/nats/subjects/responses/prompt_response.rs                   27       0  100.00%
crates/acp-nats/src/nats/subjects/responses/cancelled.rs                         18       0  100.00%
crates/acp-nats-ws/src/config.rs                                                 83       0  100.00%
crates/acp-nats-ws/src/main.rs                                                  187      18  90.37%   87, 204-225, 303
crates/acp-nats-ws/src/connection.rs                                            166      35  78.92%   75-82, 87-98, 114, 116-117, 122, 133-135, 142, 146, 150, 153-161, 172, 176, 179, 182-186, 220
crates/acp-nats-ws/src/upgrade.rs                                                57       2  96.49%   59, 90
crates/trogon-nats/src/jetstream/mocks.rs                                       474       0  100.00%
crates/acp-nats-agent/src/connection.rs                                        1434       1  99.93%   686
crates/acp-nats-stdio/src/config.rs                                              72       0  100.00%
crates/acp-nats-stdio/src/main.rs                                               141      27  80.85%   62, 114-121, 127-129, 146, 177-198
crates/acp-nats/src/nats/subjects/global/ext.rs                                  12       0  100.00%
crates/acp-nats/src/nats/subjects/global/initialize.rs                            8       0  100.00%
crates/acp-nats/src/nats/subjects/global/session_list.rs                          8       0  100.00%
crates/acp-nats/src/nats/subjects/global/session_new.rs                           8       0  100.00%
crates/acp-nats/src/nats/subjects/global/logout.rs                                8       0  100.00%
crates/acp-nats/src/nats/subjects/global/authenticate.rs                          8       0  100.00%
crates/acp-nats/src/nats/subjects/global/ext_notify.rs                           12       0  100.00%
crates/acp-nats/src/telemetry/metrics.rs                                         65       0  100.00%
crates/acp-nats/src/client/session_update.rs                                     55       0  100.00%
crates/acp-nats/src/client/rpc_reply.rs                                          71       0  100.00%
crates/acp-nats/src/client/terminal_release.rs                                  357       0  100.00%
crates/acp-nats/src/client/mod.rs                                              2987       0  100.00%
crates/acp-nats/src/client/terminal_wait_for_exit.rs                            396       0  100.00%
crates/acp-nats/src/client/fs_write_text_file.rs                                451       0  100.00%
crates/acp-nats/src/client/ext.rs                                               365       8  97.81%   193-204, 229-240
crates/acp-nats/src/client/fs_read_text_file.rs                                 384       0  100.00%
crates/acp-nats/src/client/terminal_create.rs                                   294       0  100.00%
crates/acp-nats/src/client/ext_session_prompt_response.rs                       157       0  100.00%
crates/acp-nats/src/client/terminal_kill.rs                                     309       0  100.00%
crates/acp-nats/src/client/terminal_output.rs                                   223       0  100.00%
crates/acp-nats/src/client/request_permission.rs                                338       0  100.00%
crates/acp-telemetry/src/log.rs                                                  70       2  97.14%   39-40
crates/acp-telemetry/src/trace.rs                                                32       4  87.50%   23-24, 31-32
crates/acp-telemetry/src/signal.rs                                                3       3  0.00%    4-43
crates/acp-telemetry/src/metric.rs                                               35       4  88.57%   30-31, 38-39
crates/acp-telemetry/src/service_name.rs                                         20       0  100.00%
crates/acp-telemetry/src/lib.rs                                                 153      22  85.62%   39-46, 81, 86, 91, 105-120
crates/acp-nats/src/nats/parsing.rs                                             285       1  99.65%   153
crates/acp-nats/src/nats/extensions.rs                                            3       0  100.00%
crates/acp-nats/src/nats/mod.rs                                                  23       0  100.00%
crates/trogon-source-github/src/main.rs                                           4       0  100.00%
crates/trogon-source-github/src/config.rs                                       104       0  100.00%
crates/trogon-source-github/src/server.rs                                       390       0  100.00%
crates/trogon-source-github/src/signature.rs                                     64       0  100.00%
crates/trogon-std/src/json.rs                                                    30       0  100.00%
crates/trogon-std/src/args.rs                                                    10       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/prompt_wildcard.rs               11       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/global_all.rs                    11       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/all_agent_ext.rs                 11       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/all_agent.rs                     11       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/one_agent.rs                     18       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/one_session.rs                   18       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/all_session.rs                   11       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/all_client.rs                    11       0  100.00%
crates/acp-nats/src/nats/subjects/subscriptions/one_client.rs                    18       0  100.00%
crates/acp-nats/src/agent/resume_session.rs                                     102       0  100.00%
crates/acp-nats/src/agent/set_session_mode.rs                                    71       0  100.00%
crates/acp-nats/src/agent/ext_notification.rs                                    88       0  100.00%
crates/acp-nats/src/agent/load_session.rs                                       101       0  100.00%
crates/acp-nats/src/agent/prompt.rs                                             633       0  100.00%
crates/acp-nats/src/agent/logout.rs                                              49       0  100.00%
crates/acp-nats/src/agent/set_session_model.rs                                   71       0  100.00%
crates/acp-nats/src/agent/js_request.rs                                         304       0  100.00%
crates/acp-nats/src/agent/test_support.rs                                       299       0  100.00%
crates/acp-nats/src/agent/authenticate.rs                                        52       0  100.00%
crates/acp-nats/src/agent/cancel.rs                                             105       0  100.00%
crates/acp-nats/src/agent/new_session.rs                                         91       0  100.00%
crates/acp-nats/src/agent/fork_session.rs                                       106       0  100.00%
crates/acp-nats/src/agent/mod.rs                                                 65       0  100.00%
crates/acp-nats/src/agent/set_session_config_option.rs                           71       0  100.00%
crates/acp-nats/src/agent/list_sessions.rs                                       50       0  100.00%
crates/acp-nats/src/agent/bridge.rs                                             123       4  96.75%   109-112
crates/acp-nats/src/agent/close_session.rs                                       67       0  100.00%
crates/acp-nats/src/agent/initialize.rs                                          83       0  100.00%
crates/acp-nats/src/agent/ext_method.rs                                          92       0  100.00%
crates/acp-nats/src/nats/subjects/mod.rs                                        380       0  100.00%
crates/acp-nats/src/nats/subjects/stream.rs                                      58       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/terminal_wait_for_exit.rs           15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/fs_read_text_file.rs                15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/session_update.rs                   15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/session_request_permission.rs       15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/fs_write_text_file.rs               15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/terminal_release.rs                 15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/terminal_create.rs                  15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/terminal_kill.rs                    15       0  100.00%
crates/acp-nats/src/nats/subjects/client_ops/terminal_output.rs                  15       0  100.00%
crates/acp-nats/src/nats/subjects/commands/set_mode.rs                           18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/close.rs                              18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/fork.rs                               18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/set_model.rs                          18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/set_config_option.rs                  18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/resume.rs                             18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/prompt.rs                             18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/cancel.rs                             18       0  100.00%
crates/acp-nats/src/nats/subjects/commands/load.rs                               18       0  100.00%
crates/trogon-nats/src/auth.rs                                                  114       3  97.37%   45-47
crates/trogon-nats/src/messaging.rs                                             528       2  99.62%   132, 142
crates/trogon-nats/src/connect.rs                                                96      16  83.33%   22-24, 37, 49, 68-151
crates/trogon-nats/src/client.rs                                                 25      25  0.00%    50-89
crates/trogon-nats/src/token.rs                                                   8       0  100.00%
crates/trogon-nats/src/nats_token.rs                                            161       0  100.00%
crates/trogon-nats/src/mocks.rs                                                 304       0  100.00%
crates/trogon-std/src/dirs/fixed.rs                                              84       0  100.00%
crates/trogon-std/src/dirs/system.rs                                             98      11  88.78%   57, 65, 67, 75, 77, 85, 87, 96, 98, 109, 154
crates/trogon-std/src/fs/mem.rs                                                 220      10  95.45%   61-63, 77-79, 133-135, 158
crates/trogon-std/src/fs/system.rs                                               29      12  58.62%   17-19, 31-45
crates/acp-nats/src/jetstream/ext_policy.rs                                      26       0  100.00%
crates/acp-nats/src/jetstream/consumers.rs                                       99       0  100.00%
crates/acp-nats/src/jetstream/streams.rs                                        194       4  97.94%   254-256, 266
crates/acp-nats/src/jetstream/provision.rs                                       61       0  100.00%
crates/trogon-std/src/time/system.rs                                             24       0  100.00%
crates/trogon-std/src/time/mock.rs                                              123       0  100.00%
TOTAL                                                                         17205     214  98.76%

Diff against main

Filename                                        Stmts    Miss  Cover
--------------------------------------------  -------  ------  --------
crates/trogon-source-github/src/main.rs            +4       0  +100.00%
crates/trogon-source-github/src/config.rs        +104       0  +100.00%
crates/trogon-source-github/src/server.rs        +390       0  +100.00%
crates/trogon-source-github/src/signature.rs      +64       0  +100.00%
TOTAL                                            +562       0  +100.00%

Results for commit: 88c673d

Minimum allowed coverage is 95%

♻️ This comment has been updated with latest results

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

refactor: rename trogon-sink-github to trogon-source-github

88c673d

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis merged commit 6fdf052 into main Apr 3, 2026
6 of 7 checks passed

yordis deleted the yordis/rename-sink-to-source branch April 3, 2026 00:10

coderabbitai Bot mentioned this pull request Apr 3, 2026

feat(slack): add trogon-source-slack webhook receiver #90

Merged

yordis added a commit that referenced this pull request Jun 16, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

dcac7fb

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 22, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

698b143

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 22, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

c043d6b

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 23, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

62717c0

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 23, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

aba5c77

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 23, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

5f3a362

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 25, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

5d510b0

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 25, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

5888e0d

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 26, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

43ac3f9

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 26, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

47643d9

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 26, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

0493745

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 26, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

33ff471

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

yordis added a commit that referenced this pull request Jun 26, 2026

refactor: rename trogon-sink-github to trogon-source-github (#91)

958400d

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

Navigation Menu

Uh oh!

refactor: rename trogon-sink-github to trogon-source-github - #91

refactor: rename trogon-sink-github to trogon-source-github#91
yordis merged 1 commit into
mainfrom
yordis/rename-sink-to-source

yordis commented Apr 2, 2026

Uh oh!

cursor Bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Apr 2, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

yordis commented Apr 2, 2026

Summary

Uh oh!

cursor Bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

coderabbitai Bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Poem

Uh oh!

github-actions Bot commented Apr 2, 2026

Code Coverage Summary

Diff against main

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

cursor Bot commented Apr 2, 2026 •

edited

Loading

coderabbitai Bot commented Apr 2, 2026 •

edited

Loading