Refactoring source code to be fully a bot by apentori · Pull Request #21 · status-im/status-bot

apentori · 2026-05-20T20:23:49Z

Description

Refactoring the source code to be fully a Bot and not just a monitoring tools

Module for allowing more usage
API to interact with the messages
Metrics, because all correct application have metrics.

Left to do

Fix the monitoring part - issue with the coingechko interraction during test.
API interraction with the communities.
Add correct versioning.
Improve metrics
Add more modules like basic webhook interraction

Warning: part is AI generated, so it might be ugly / stupid, but that's a start
Note: I will squash all commit later

Signed-off-by: apentori <pentori.alexis@proton.me>

Renaming configuration for lisibility Signed-off-by: apentori <pentori.alexis@proton.me>

nickninov · 2026-05-21T06:33:34Z

Seems an overkill for the project. Looks like a Dagster Config.

Well that;s just a proper config management

nickninov · 2026-05-21T06:39:29Z

The purpose of ./bot/ folder is to convert it to a PyPi library. At the moment there are things that are related to community monitoring and status-bot.

nickninov · 2026-05-21T08:49:28Z

It seems like we are reinventing the wheel (with the help of AI) with so many new modules that are completely unnecessary. At the end of the day community monitoring is a batch processing / micro batch processing problem. The most maintainable and simple thing is to convert monitor.py to a pipeline with currently available tools. All of these abstractions are handled for us and we can just focus on the data. class Account can be turned into an Airbyte connector that communicates with status-go.

account.py is still under development and should not be touched until it is stable and reusable. We want to keep it as easy as possible for developers. The idea of a script per use case was to showcase how account.py can be used in different scenarios.

nickninov · 2026-05-21T07:28:27Z

This code was copied from another Postgres instance. We should actually centralize a class like Postgres for overall code consistency.

nickninov · 2026-05-21T07:32:18Z

At the moment this is unnecessary. listen is still in early development stages. Please revert to previous version

nickninov · 2026-05-21T07:43:00Z

This information was in the main README.md to make users aware of what we are doing. At the moment it just seems hidden and people have to look for this.

nickninov · 2026-05-21T07:48:24Z

-    # Public information for the bot
-    public_key: "0x041658626a9e1303b631f6d0fb1e047211d5603b977454f7d5d29fe583c3d6c1bd3d8e395d67f6c44b5bc659aae912040e9dd8164b5107368a29029cb53389d8b0"
-    compressed_key: "zQ3shNv1tnajHo5FvCvP662cWcbBfS5ZejB4TWaH9iAuFCZZe"


This information is crucial and should not be deleted. This makes the bot in the communities as transparent. Users can verify if they are communicating with the official Status bot.

nickninov · 2026-05-21T08:57:00Z

+    if display_name in available_accounts:
+        account.logger.info(f"Logging in with display name: {display_name}")
+        account.login(
+            display_name=display_name,
+            password=password,
+            infura_token=config.bot.infura_token,
+            coingecko_api_key=config.bot.coingecko_api_key
+        )
+    elif config.bot.init_account:
+        mnemonic = config.bot.mnemonic_phrase
+        if not mnemonic:
+            raise ValueError(
+                "init_account is true but no mnemonic_phrase provided"
+            )
+        account.logger.info(f"Creating/restoring account: {display_name}")
+        account.login(
+            display_name=display_name,
+            password=password,
+            mnemonic=mnemonic,
+            infura_token=config.bot.infura_token,
+            coingecko_api_key=config.bot.coingecko_api_key
+        )
+    else:
+        raise ValueError(
+            f"Account '{display_name}' not found and init_account is false. "
+            f"Available accounts: {[a['display_name'] for a in available_accounts]}"
+        )


This is reinventing account.login. These checks are already implemented in the method.

nickninov · 2026-05-21T13:55:35Z

API interraction with the communities.

Please refer to status-im/status-python-sdk#5. As described in the issue, there are blockers.

apentori · 2026-05-22T11:32:03Z

It seems like we are reinventing the wheel (with the help of AI) with so many new modules that are completely unnecessary. At the end of the day community monitoring is a batch processing / micro batch processing problem.

This repo is not just for monitoring the community, else the repo would be call status-community-monitoring

nickninov · 2026-05-22T11:47:11Z

@apentori we can keep the repository as it is and just migrate bot and its documentation. Can make a repo like status-python-sdk.

nickninov · 2026-05-25T10:22:47Z

SDK functionality has been moved to status-im/status-python-sdk.

Signed-off-by: apentori <pentori.alexis@proton.me>

apentori added 10 commits May 15, 2026 19:18

add refactoring plan

387c7da

Signed-off-by: apentori <pentori.alexis@proton.me>

refacto v1

f95224e

Signed-off-by: apentori <pentori.alexis@proton.me>

fix code

7f045c9

Signed-off-by: apentori <pentori.alexis@proton.me>

refacto: extract config and metrics

541de4d

Signed-off-by: apentori <pentori.alexis@proton.me>

fix: improve config

9e7cb0f

Signed-off-by: apentori <pentori.alexis@proton.me>

tmp: adding monitoring as module

ba34006

Signed-off-by: apentori <pentori.alexis@proton.me>

fix module loading

82226ff

Signed-off-by: apentori <pentori.alexis@proton.me>

docs: small update

3ff1971

Signed-off-by: apentori <pentori.alexis@proton.me>

feat: api server

47a561e

Signed-off-by: apentori <pentori.alexis@proton.me>

docs: update Documentation

19091a1

Renaming configuration for lisibility Signed-off-by: apentori <pentori.alexis@proton.me>

apentori requested review from nickninov and siddarthkay May 20, 2026 20:23

apentori self-assigned this May 20, 2026

apentori changed the title ~~Refactoring to setup api, metrics and allow modules to be loaded~~ Refactoring source code to be fully a bot May 20, 2026

nickninov reviewed May 21, 2026

View reviewed changes

Comment thread bot/modules/messaging.py

nickninov reviewed May 21, 2026

View reviewed changes

Comment thread bot/account.py

apentori added 3 commits May 27, 2026 19:35

api: adding optional API KEY

c28a596

Signed-off-by: apentori <pentori.alexis@proton.me>

api: add wehbook

c5afc4d

Signed-off-by: apentori <pentori.alexis@proton.me>

refactor database and add receiver modules

ffc525f

Signed-off-by: apentori <pentori.alexis@proton.me>

Conversation

apentori commented May 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Left to do

Uh oh!

nickninov May 21, 2026

Choose a reason for hiding this comment

Uh oh!

apentori May 21, 2026

Choose a reason for hiding this comment

Uh oh!

nickninov commented May 21, 2026

Uh oh!

Uh oh!

nickninov commented May 21, 2026

Uh oh!

nickninov May 21, 2026

Choose a reason for hiding this comment

Uh oh!

nickninov May 21, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

nickninov May 21, 2026

Choose a reason for hiding this comment

Uh oh!

nickninov May 21, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

nickninov May 21, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nickninov commented May 21, 2026

Uh oh!

apentori commented May 22, 2026

Uh oh!

nickninov commented May 22, 2026

Uh oh!

nickninov commented May 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

apentori commented May 20, 2026 •

edited

Loading