Diff channels on PRs

Author: raccube

.github/workflows/diff-channels.yml

@@ -0,0 +1,47 @@
+name: Diff channels
+on:
+  pull_request:
+    paths:
+      - channels.yml
+      - channels.schema.yml
+jobs:
+  diff-channels:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/checkout@v4
+        with:
+          ref: ${{ github.base_ref }}
+          path: pr-base
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: "3.11"
+
+      - name: Cache dependencies
+        uses: actions/cache@v3
+        id: cache-dep
+        with:
+          path: venv
+          key: deps-1-${{ hashfiles('script/requirements.txt') }}
+
+      - name: Install Dependencies
+        run: |
+          python3 -m venv venv
+          venv/bin/pip install -U pip wheel
+          venv/bin/pip install -r script/requirements.txt
+          venv/bin/pip install .
+
+      - name: Diff channels
+        run: |
+          source venv/bin/activate
+          python -m sr.discord_bot diff pr-base/channels.yml channels.yml > /tmp/channel-diff.txt
+
+      - name: Post diff as comment on PR
+        uses: thollander/actions-comment-pull-request@v3
+        with:
+          file-path: /tmp/channel-diff.txt
+          comment-tag: execution

channels.yml

@@ -111,6 +111,7 @@
     - name: community-updates
       topic: Discord community updates
       use_case: discord
+    - name: ci-test
 
 # Do not change the category below. It is populated automatically when users join.
 - name: Welcome

src/sr/discord_bot/__main__.py

@@ -4,10 +4,13 @@
 import argparse
 
 import sentry_sdk
+import yaml
 from dotenv import load_dotenv
 from discord import Intents
 
 from sr.discord_bot.bot import BotClient
+from sr.discord_bot.channel import ChannelSet
+from sr.discord_bot.schema import ChannelDefinition
 
 load_dotenv()
 logger = logging.getLogger("srbot")
@@ -20,24 +23,42 @@
     traces_sample_rate=1.0,
 )
 
-intents = Intents.default()
-intents.members = True  # Listen to member joins
-
-token = os.getenv("DISCORD_TOKEN")
-if token is None:
-    print("No token provided.", file=sys.stderr)
-    exit(1)
-
 parser = argparse.ArgumentParser(description="Student Robotics Discord Bot")
 subcommands = parser.add_subparsers(dest="command")
 parser_run = subcommands.add_parser("run", help="Run the Discord bot")
 parser_plan = subcommands.add_parser("plan", help="List pending guild changes")
 parser_apply = subcommands.add_parser("apply", help="Apply pending guild changes")
+parser_diff = subcommands.add_parser("diff", help="Compare two channel configurations")
+parser_diff.add_argument("old_config", type=argparse.FileType('r'))
+parser_diff.add_argument("new_config", type=argparse.FileType('r'))
+
 args = parser.parse_args()
 
 if args.command is None:
     parser.print_help()
     exit(1)
+elif args.command == "diff":
+    # The diff command doesn't require us to connect to Discord so we can handle it here
+    if args.old_config is None or args.new_config is None:
+        parser.print_help()
+        exit(1)
+    old = [ChannelDefinition.load(ch) for ch in yaml.load(args.old_config.read(), Loader=yaml.Loader)]
+    new = [ChannelDefinition.load(ch) for ch in yaml.load(args.new_config.read(), Loader=yaml.Loader)]
+    diff = ChannelSet.diff(old, new)
+    if len(diff) > 0:
+        for change in diff:
+            print(change)
+    else:
+        print("No changes found.")
+    exit(0)
+
+intents = Intents.default()
+intents.members = True  # Listen to member joins
+
+token = os.getenv("DISCORD_TOKEN")
+if token is None:
+    print("No token provided.", file=sys.stderr)
+    exit(1)
 
 bot = BotClient(logger=logger, intents=intents)
 bot.mode = args.command