Skip to content

[Misc] Misc fixes & improvements#1632

Merged
Olshansk merged 4 commits intomainfrom
misc_fixes
Jul 3, 2025
Merged

[Misc] Misc fixes & improvements#1632
Olshansk merged 4 commits intomainfrom
misc_fixes

Conversation

@Olshansk
Copy link
Copy Markdown
Collaborator

@Olshansk Olshansk commented Jul 3, 2025 •
edited
Loading

Handful of small, different, miscleanious changes:

  • Test mode to Telegram workflows: Parameterized telegram-send-message.yml, telegram-broadcast.yml, and telegram-notify-release.yml with a test_mode boolean flag that sends messages only to the testing group when enabled.
  • Refactor build tools organization: Moved proof_payload_size_analysis target from query.mk to a new tools.mk file and included it in the main Makefile for better separation of concerns.
  • Enhance test notifications: Test messages are clearly marked with "🧪 TEST" prefix to distinguish them from production broadcasts in the Telegram testing group.
  • Update a few rand.Read in tests with //nolint:staticcheck // Using rand.Read in tests as a deterministic pseudo-random source is okay.
  • Fixed the main build
  • Makefile telegram commands for testing: Created these commands to safely test Telegram notifications before broadcasting.
`make telegram_test_message MSG="..."`
`make telegram_test_release`

@Olshansk Olshansk self-assigned this Jul 3, 2025
@Olshansk Olshansk added code health Cleans up some code tooling Tooling - CLI, scripts, helpers, off-chain, etc... labels Jul 3, 2025
@Olshansk Olshansk added this to Shannon Jul 3, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Shannon Jul 3, 2025
@github-actions github-actions bot added the consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. label Jul 3, 2025
@Olshansk Olshansk moved this from 📋 Backlog to 🏗 In progress in Shannon Jul 3, 2025
@Olshansk Olshansk moved this from 🏗 In progress to 👀 In review in Shannon Jul 3, 2025
@Olshansk Olshansk marked this pull request as ready for review July 3, 2025 20:31
@Olshansk Olshansk merged commit e203288 into main Jul 3, 2025
11 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Shannon Jul 3, 2025
bryanchriswhite added a commit that referenced this pull request Jul 16, 2025
@bryanchriswhite
Merge remote-tracking branch 'pokt/main' into chore/ibc
fbff8bc 
* pokt/main: (69 commits)
  Refactor Tiltfile structure and enhance Pocketdex support. (#1640)
  [Morse Recovery] Updated Recovery Allowlist (#1651)
  [RelayMiner] Improve SafeReadBody concurrency safety by copying buffer data (#1650)
  Add a configurable max body size parameter to RelayMiner configuration. (#1566)
  Update upgrade_tx_v0.1.25_alpha.json
  feat: Improve relay request unmarshaling error handling and logging (#1649)
  [Docs] v0.1.25/26 upgrage artifacts (#1648)
  Remove settlement result from `EventClaimSettled` in `v0.127` release. (#1647)
  Revert "feat: Remove settlement result from EventClaimSettled" (#1643)
  [Code Health] Fix linter issues & attempt to fix flaky tests (#1646)
  Updating config-path-x.yaml with: schema header, owned app for config-path-1
  [Shannon SDK] Update the `shannon-sdk` in prep for `v0.1.26` (#1641)
  [RelayMiner] Add chain version retrieval functionality to block client (#1639)
  [RelayMiner] Avoid concurrent access to shared logger (#1636)
  [Telegram] Add Korbit channel to broadcast workflow (#1634)
  upgrade: Prepare upgrade plan for v0.1.26 (#1635)
  [RelayMiner] Add proper timeout handling for RelayMiner HTTP requests (#1595)
  [Code only] fix: implement backward compatible relay response signatures  (from #1629) (#1630)
  [Misc] Misc fixes & improvements (#1632)
  Added pocket-knife to ecosystem tools (#1622)
  ...
bryanchriswhite added a commit that referenced this pull request Jul 16, 2025
@bryanchriswhite
Merge branch 'refs/heads/chore/ibc' into chore/localnet/tilt
e93c96b 
* refs/heads/chore/ibc: (69 commits)
  Refactor Tiltfile structure and enhance Pocketdex support. (#1640)
  [Morse Recovery] Updated Recovery Allowlist (#1651)
  [RelayMiner] Improve SafeReadBody concurrency safety by copying buffer data (#1650)
  Add a configurable max body size parameter to RelayMiner configuration. (#1566)
  Update upgrade_tx_v0.1.25_alpha.json
  feat: Improve relay request unmarshaling error handling and logging (#1649)
  [Docs] v0.1.25/26 upgrage artifacts (#1648)
  Remove settlement result from `EventClaimSettled` in `v0.127` release. (#1647)
  Revert "feat: Remove settlement result from EventClaimSettled" (#1643)
  [Code Health] Fix linter issues & attempt to fix flaky tests (#1646)
  Updating config-path-x.yaml with: schema header, owned app for config-path-1
  [Shannon SDK] Update the `shannon-sdk` in prep for `v0.1.26` (#1641)
  [RelayMiner] Add chain version retrieval functionality to block client (#1639)
  [RelayMiner] Avoid concurrent access to shared logger (#1636)
  [Telegram] Add Korbit channel to broadcast workflow (#1634)
  upgrade: Prepare upgrade plan for v0.1.26 (#1635)
  [RelayMiner] Add proper timeout handling for RelayMiner HTTP requests (#1595)
  [Code only] fix: implement backward compatible relay response signatures  (from #1629) (#1630)
  [Misc] Misc fixes & improvements (#1632)
  Added pocket-knife to ecosystem tools (#1622)
  ...

# Conflicts:
#	Makefile
#	Tiltfile
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bryanchriswhite bryanchriswhite Awaiting requested review from bryanchriswhite

@ArtSabintsev ArtSabintsev Awaiting requested review from ArtSabintsev

Assignees

@Olshansk Olshansk

Labels

code health Cleans up some code consensus-breaking IMPORTANT! If the PR with this tag is merged, next release WILL HAVE TO BE an upgrade. tooling Tooling - CLI, scripts, helpers, off-chain, etc...

Projects

Shannon
Status: ✅ Done

Milestone

RelayMiner Stability

Development

Successfully merging this pull request may close these issues.

1 participant

@Olshansk