Compare commits
5 Commits
235a89653f
...
feature/ca
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
e43c36b84f | ||
| 44680dcb4c | |||
|
|
7c17bff110 | ||
| 43d07eae92 | |||
|
|
66d2a9fb86 |
@@ -11,7 +11,7 @@ import os
|
||||
import tempfile
|
||||
from pathlib import Path
|
||||
|
||||
from core.models import Bounty, RoomData, TrackingData, TrackedBounty
|
||||
from core.models import Bounty, Category, RoomData, TrackingData, TrackedBounty
|
||||
|
||||
|
||||
class JsonFileRoomStorage:
|
||||
@@ -57,16 +57,28 @@ class JsonFileRoomStorage:
|
||||
created_by_user_id=b["created_by_user_id"],
|
||||
deleted_at=b.get("deleted_at"),
|
||||
created_by_username=b.get("created_by_username"),
|
||||
category_ids=b.get("category_ids", []),
|
||||
)
|
||||
for b in data.get("bounties", [])
|
||||
]
|
||||
|
||||
categories = [
|
||||
Category(
|
||||
id=c["id"],
|
||||
name=c["name"],
|
||||
created_at=c["created_at"],
|
||||
deleted_at=c.get("deleted_at"),
|
||||
)
|
||||
for c in data.get("categories", [])
|
||||
]
|
||||
|
||||
return RoomData(
|
||||
room_id=data["room_id"],
|
||||
bounties=bounties,
|
||||
next_id=data["next_id"],
|
||||
timezone=data.get("timezone"),
|
||||
admin_usernames=data.get("admin_usernames", []),
|
||||
categories=categories,
|
||||
)
|
||||
|
||||
def save(self, room_data: RoomData) -> None:
|
||||
@@ -76,6 +88,15 @@ class JsonFileRoomStorage:
|
||||
"next_id": room_data.next_id,
|
||||
"timezone": room_data.timezone,
|
||||
"admin_usernames": room_data.admin_usernames or [],
|
||||
"categories": [
|
||||
{
|
||||
"id": c.id,
|
||||
"name": c.name,
|
||||
"created_at": c.created_at,
|
||||
"deleted_at": c.deleted_at,
|
||||
}
|
||||
for c in room_data.categories
|
||||
],
|
||||
"bounties": [
|
||||
{
|
||||
"id": b.id,
|
||||
@@ -86,6 +107,7 @@ class JsonFileRoomStorage:
|
||||
"created_by_user_id": b.created_by_user_id,
|
||||
"deleted_at": b.deleted_at,
|
||||
"created_by_username": b.created_by_username,
|
||||
"category_ids": b.category_ids,
|
||||
}
|
||||
for b in room_data.bounties
|
||||
],
|
||||
|
||||
@@ -15,6 +15,7 @@ from commands import (
|
||||
cmd_add,
|
||||
cmd_admin,
|
||||
cmd_bounty,
|
||||
cmd_category,
|
||||
cmd_delete,
|
||||
cmd_delete_message,
|
||||
cmd_edit,
|
||||
@@ -61,6 +62,7 @@ def build_app() -> Application:
|
||||
app.add_handler(CommandHandler("timezone", cmd_timezone))
|
||||
app.add_handler(CommandHandler("admin", cmd_admin))
|
||||
app.add_handler(CommandHandler("recover", cmd_recover))
|
||||
app.add_handler(CommandHandler("category", cmd_category))
|
||||
|
||||
app.add_handler(CallbackQueryHandler(cmd_delete_message))
|
||||
|
||||
@@ -82,6 +84,7 @@ async def post_init(app: Application) -> None:
|
||||
("timezone", "Get/set room timezone"),
|
||||
("admin", "Manage admins"),
|
||||
("recover", "Recover deleted bounties"),
|
||||
("category", "Manage categories"),
|
||||
("help", "Show help"),
|
||||
]
|
||||
)
|
||||
|
||||
@@ -60,12 +60,19 @@ def extract_args(text: str) -> list[str]:
|
||||
def parse_args(
|
||||
args: list[str],
|
||||
timezone_str: str = "UTC",
|
||||
) -> tuple[Optional[str], Optional[str], Optional[int], bool, bool]:
|
||||
) -> tuple[Optional[str], Optional[str], Optional[int], bool, bool, list[str], bool]:
|
||||
"""Parse command arguments.
|
||||
|
||||
Returns:
|
||||
(text, link, due_date_ts, clear_link, clear_date, category_slugs, clear_categories)
|
||||
"""
|
||||
text = None
|
||||
link = None
|
||||
due_date_ts = None
|
||||
clear_link = False
|
||||
clear_date = False
|
||||
category_slugs: list[str] = []
|
||||
clear_categories = False
|
||||
|
||||
try:
|
||||
tz = ZoneInfo(timezone_str)
|
||||
@@ -109,6 +116,10 @@ def parse_args(
|
||||
return int(localized.timestamp())
|
||||
return None
|
||||
|
||||
def parse_category_slugs(cat_str: str) -> list[str]:
|
||||
"""Parse comma-separated category slugs."""
|
||||
return [s.strip().lower() for s in cat_str.split(",") if s.strip()]
|
||||
|
||||
i = 0
|
||||
while i < len(args):
|
||||
arg = args[i]
|
||||
@@ -137,6 +148,16 @@ def parse_args(
|
||||
else:
|
||||
clear_date = True
|
||||
i += 1
|
||||
elif arg == "-cat":
|
||||
if i + 1 < len(args) and args[i + 1] == "-":
|
||||
clear_categories = True
|
||||
i += 2
|
||||
elif i + 1 < len(args) and not args[i + 1].startswith("-"):
|
||||
category_slugs = parse_category_slugs(args[i + 1])
|
||||
i += 2
|
||||
else:
|
||||
clear_categories = True
|
||||
i += 1
|
||||
elif not link and is_url(arg):
|
||||
link = normalize_url(arg)
|
||||
i += 1
|
||||
@@ -161,7 +182,7 @@ def parse_args(
|
||||
else:
|
||||
text = text + " " + arg
|
||||
|
||||
return text, link, due_date_ts, clear_link, clear_date
|
||||
return text, link, due_date_ts, clear_link, clear_date, category_slugs, clear_categories
|
||||
|
||||
|
||||
def format_bounty(
|
||||
@@ -267,8 +288,23 @@ async def cmd_bounty(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
show_all = "all" in args
|
||||
args = [a for a in args if a != "all"]
|
||||
|
||||
# Parse -c flag for category filter
|
||||
category_slugs: list[str] = []
|
||||
filtered_args = []
|
||||
i = 0
|
||||
while i < len(args):
|
||||
if args[i] == "-c":
|
||||
if i + 1 < len(args) and not args[i + 1].startswith("-"):
|
||||
category_slugs = [s.strip().lower() for s in args[i + 1].split(",")]
|
||||
i += 2
|
||||
else:
|
||||
i += 1
|
||||
else:
|
||||
filtered_args.append(args[i])
|
||||
i += 1
|
||||
|
||||
try:
|
||||
limit = int(args[0]) if args else 5
|
||||
limit = int(filtered_args[0]) if filtered_args else 5
|
||||
except (ValueError, IndexError):
|
||||
limit = 5
|
||||
|
||||
@@ -286,6 +322,13 @@ async def cmd_bounty(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
return (1, b.created_at)
|
||||
|
||||
filtered_bounties = [b for b in all_bounties if not is_expired(b) or show_all]
|
||||
|
||||
# Filter by category if specified
|
||||
if category_slugs:
|
||||
filtered_bounties = [
|
||||
b for b in filtered_bounties if any(cat in b.category_ids for cat in category_slugs)
|
||||
]
|
||||
|
||||
filtered_bounties.sort(key=sort_key)
|
||||
|
||||
total_count = len(filtered_bounties)
|
||||
@@ -296,7 +339,12 @@ async def cmd_bounty(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
if show_all:
|
||||
if category_slugs:
|
||||
await update.message.reply_text(
|
||||
f"No bounties with category: {', '.join(category_slugs)}",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
elif show_all:
|
||||
await update.message.reply_text(
|
||||
"No bounties yet.",
|
||||
reply_markup=reply_markup,
|
||||
@@ -309,6 +357,9 @@ async def cmd_bounty(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
return
|
||||
|
||||
lines = []
|
||||
if category_slugs:
|
||||
lines.append(f"📂 Filtering with {', '.join(category_slugs)} categories:")
|
||||
|
||||
if limit < total_count:
|
||||
lines.append(f"Showing {limit} of {total_count} bounties:")
|
||||
slice_length = 40
|
||||
@@ -365,8 +416,8 @@ async def cmd_add(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
args = extract_args(update.message.text)
|
||||
if not args:
|
||||
await update.message.reply_text(
|
||||
"Usage: /add <text> [link] [due_date]\n"
|
||||
"Example: /add Fix the bug https://github.com/foo/bar tomorrow"
|
||||
"Usage: /add <text> [link] [due_date] [-cat <slug>[,<slug>]]\n"
|
||||
"Example: /add Fix the bug https://github.com/foo/bar tomorrow -cat bug,urgent"
|
||||
)
|
||||
return
|
||||
|
||||
@@ -377,7 +428,7 @@ async def cmd_add(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
effective_user = update.effective_user
|
||||
username = effective_user.username or effective_user.first_name or None
|
||||
|
||||
text, link, due_date_ts, _, _ = parse_args(args, timezone_str)
|
||||
text, link, due_date_ts, _, _, category_slugs, _ = parse_args(args, timezone_str)
|
||||
if not text and not link:
|
||||
await update.message.reply_text("A bounty needs at least text or a link.")
|
||||
return
|
||||
@@ -392,6 +443,18 @@ async def cmd_add(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
due_date_ts=due_date_ts,
|
||||
created_by_username=username,
|
||||
)
|
||||
|
||||
# Add categories if specified
|
||||
for slug in category_slugs:
|
||||
try:
|
||||
BOUNTY_SERVICE.add_category_to_bounty(
|
||||
room_id, bounty.id, slug, username
|
||||
)
|
||||
except ValueError:
|
||||
pass # Category doesn't exist, skip it
|
||||
except PermissionError:
|
||||
pass
|
||||
|
||||
except PermissionError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
@@ -411,10 +474,15 @@ async def cmd_add(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
if due_date_ts:
|
||||
timezone_str = BOUNTY_SERVICE.get_timezone(room_id)
|
||||
due_str = f" | Due: {format_due_date(due_date_ts, timezone_str)}"
|
||||
|
||||
cat_str = ""
|
||||
if category_slugs:
|
||||
cat_str = f" | 📂 {', '.join(category_slugs)}"
|
||||
|
||||
keyboard = [[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
f"✅ Bounty added (#{bounty.id}){due_str}",
|
||||
f"✅ Bounty added (#{bounty.id}){due_str}{cat_str}",
|
||||
disable_web_page_preview=True,
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
@@ -443,11 +511,15 @@ async def cmd_update(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
"Usage: /update <bounty_id> [text] [link] [due_date]\n"
|
||||
" /update <bounty_id> -link [<url>] - clear or set link\n"
|
||||
" /update <bounty_id> -date [<date>] - clear or set date\n"
|
||||
" /update <bounty_id> -cat <slug>[,<slug>] - add/replace categories\n"
|
||||
" /update <bounty_id> -cat - - clear categories\n"
|
||||
" /update <bounty_id> -remove-cat <slug> - remove category\n"
|
||||
"Examples:\n"
|
||||
" /update 1 new text - update text only\n"
|
||||
" /update 1 -link - clear link\n"
|
||||
" /update 1 -link https://... - set link\n"
|
||||
" /update 1 -link -date - clear both link and date"
|
||||
" /update 1 -cat bug,feature - set categories\n"
|
||||
" /update 1 -remove-cat bug - remove category"
|
||||
)
|
||||
return
|
||||
|
||||
@@ -466,23 +538,38 @@ async def cmd_update(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
|
||||
timezone_str = BOUNTY_SERVICE.get_timezone(room_id)
|
||||
|
||||
text, link, due_date_ts, clear_link, clear_date = parse_args(args[1:], timezone_str)
|
||||
if (
|
||||
not text
|
||||
and not link
|
||||
and due_date_ts is None
|
||||
and not clear_link
|
||||
and not clear_date
|
||||
):
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"Nothing to update.",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
# Parse flags separately to handle -cat and -remove-cat specially
|
||||
remaining_args = args[1:]
|
||||
category_slugs: list[str] = []
|
||||
clear_categories = False
|
||||
remove_category_slug: str | None = None
|
||||
|
||||
# Process remaining args for special flags
|
||||
filtered_args = []
|
||||
i = 0
|
||||
while i < len(remaining_args):
|
||||
arg = remaining_args[i]
|
||||
if arg == "-cat":
|
||||
if i + 1 < len(remaining_args) and remaining_args[i + 1] == "-":
|
||||
clear_categories = True
|
||||
i += 2
|
||||
elif i + 1 < len(remaining_args) and not remaining_args[i + 1].startswith("-"):
|
||||
category_slugs = [s.strip().lower() for s in remaining_args[i + 1].split(",")]
|
||||
i += 2
|
||||
else:
|
||||
clear_categories = True
|
||||
i += 1
|
||||
elif arg == "-remove-cat":
|
||||
if i + 1 < len(remaining_args) and not remaining_args[i + 1].startswith("-"):
|
||||
remove_category_slug = remaining_args[i + 1].lower()
|
||||
i += 2
|
||||
else:
|
||||
i += 1
|
||||
else:
|
||||
filtered_args.append(arg)
|
||||
i += 1
|
||||
|
||||
text, link, due_date_ts, clear_link, clear_date, _, _ = parse_args(filtered_args, timezone_str)
|
||||
|
||||
old_bounty = BOUNTY_SERVICE.get_bounty(room_id, bounty_id)
|
||||
if not old_bounty:
|
||||
@@ -496,8 +583,34 @@ async def cmd_update(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
)
|
||||
return
|
||||
|
||||
changes = []
|
||||
|
||||
# Handle category operations
|
||||
if clear_categories:
|
||||
try:
|
||||
success = BOUNTY_SERVICE.update_bounty(
|
||||
BOUNTY_SERVICE.update_bounty_categories(room_id, bounty_id, [], username)
|
||||
changes.append("Categories cleared")
|
||||
except (PermissionError, ValueError):
|
||||
pass
|
||||
|
||||
if category_slugs:
|
||||
try:
|
||||
BOUNTY_SERVICE.update_bounty_categories(room_id, bounty_id, category_slugs, username)
|
||||
changes.append(f"Categories set: {', '.join(category_slugs)}")
|
||||
except (PermissionError, ValueError):
|
||||
pass
|
||||
|
||||
if remove_category_slug:
|
||||
try:
|
||||
BOUNTY_SERVICE.remove_category_from_bounty(room_id, bounty_id, remove_category_slug, username)
|
||||
changes.append(f"Removed category: {remove_category_slug}")
|
||||
except (PermissionError, ValueError):
|
||||
pass
|
||||
|
||||
# Only update other fields if something was provided
|
||||
if text is not None or link is not None or due_date_ts is not None or clear_link or clear_date:
|
||||
try:
|
||||
BOUNTY_SERVICE.update_bounty(
|
||||
room_id=room_id,
|
||||
bounty_id=bounty_id,
|
||||
username=username,
|
||||
@@ -507,23 +620,6 @@ async def cmd_update(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
clear_link=clear_link,
|
||||
clear_due=clear_date,
|
||||
)
|
||||
except PermissionError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
return
|
||||
except ValueError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
return
|
||||
|
||||
if success:
|
||||
changes = []
|
||||
if text is not None:
|
||||
old_text = old_bounty.text or "(none)"
|
||||
changes.append(f"Text: {old_text} → {text}")
|
||||
@@ -548,6 +644,20 @@ async def cmd_update(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
else "(none)"
|
||||
)
|
||||
changes.append(f"Date: {old_date} → (cleared)")
|
||||
except PermissionError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
return
|
||||
except ValueError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
return
|
||||
|
||||
if changes:
|
||||
keyboard = [
|
||||
@@ -567,8 +677,6 @@ async def cmd_update(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
f"✅ Bounty #{bounty_id} updated.",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
else:
|
||||
await update.message.reply_text("Bounty not found.")
|
||||
|
||||
|
||||
cmd_edit = cmd_update
|
||||
@@ -842,6 +950,9 @@ async def cmd_show(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
due_str = dt_due.strftime("%d %B %Y %H:%M")
|
||||
lines.append(f"📅 {due_str} ({timezone_str})")
|
||||
|
||||
if bounty.category_ids:
|
||||
lines.append(f"📂 Categories: {' | '.join(bounty.category_ids)}")
|
||||
|
||||
if bounty.created_by_username:
|
||||
lines.append(
|
||||
f'👤 <a href="tg://user?id={bounty.created_by_user_id}">{bounty.created_by_username}</a>'
|
||||
@@ -1101,6 +1212,162 @@ async def cmd_admin(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
|
||||
|
||||
async def cmd_category(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
"""Handle /category command for listing, adding, and deleting categories."""
|
||||
user_id = get_user_id(update)
|
||||
room_id = get_room_id(update)
|
||||
effective_user = update.effective_user
|
||||
username = effective_user.username or effective_user.first_name or None
|
||||
|
||||
args = extract_args(update.message.text)
|
||||
|
||||
if not args:
|
||||
# List categories
|
||||
categories = BOUNTY_SERVICE.list_categories(room_id)
|
||||
if not categories:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"No categories yet.\n"
|
||||
"Usage: /category add <slug> <name>",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
|
||||
lines = ["Categories:"]
|
||||
for cat in categories:
|
||||
lines.append(f"- {cat.id} → {cat.name}")
|
||||
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"\n".join(lines),
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
|
||||
subcommand = args[0]
|
||||
|
||||
if subcommand == "add":
|
||||
# Add category
|
||||
if not BOUNTY_SERVICE.is_admin(room_id, username):
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"⛔ Only admins can add categories.",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
|
||||
if len(args) < 3:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"Usage: /category add <slug> <name>\n"
|
||||
"Example: /category add bug 'Bug Report'",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
|
||||
slug = args[1].lower()
|
||||
name = " ".join(args[2:])
|
||||
|
||||
try:
|
||||
category = BOUNTY_SERVICE.add_category(room_id, slug, name, username)
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
f"✅ Category added: {category.id} → {category.name}",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
except PermissionError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
except ValueError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
return
|
||||
|
||||
elif subcommand == "delete":
|
||||
# Delete category
|
||||
if not BOUNTY_SERVICE.is_admin(room_id, username):
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"⛔ Only admins can delete categories.",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
|
||||
if len(args) < 2:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"Usage: /category delete <slug>",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
return
|
||||
|
||||
slug = args[1].lower()
|
||||
|
||||
try:
|
||||
success = BOUNTY_SERVICE.delete_category(room_id, slug, username)
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
if success:
|
||||
await update.message.reply_text(
|
||||
f"✅ Category '{slug}' deleted.",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
else:
|
||||
await update.message.reply_text(
|
||||
f"⛔ Category '{slug}' not found.",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
except PermissionError as e:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(f"⛔ {e}", reply_markup=reply_markup)
|
||||
return
|
||||
|
||||
else:
|
||||
keyboard = [
|
||||
[InlineKeyboardButton("🗑️ Delete", callback_data=f"del_msg:{user_id}")]
|
||||
]
|
||||
reply_markup = InlineKeyboardMarkup(keyboard)
|
||||
await update.message.reply_text(
|
||||
"Usage:\n"
|
||||
"/category - list categories\n"
|
||||
"/category add <slug> <name> - add category (admin)\n"
|
||||
"/category delete <slug> - delete category (admin)",
|
||||
reply_markup=reply_markup,
|
||||
)
|
||||
|
||||
|
||||
async def cmd_help(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
user_id = get_user_id(update)
|
||||
room_id = get_room_id(update)
|
||||
@@ -1118,6 +1385,11 @@ async def cmd_help(update: Update, ctx: ContextTypes.DEFAULT_TYPE) -> None:
|
||||
"/delete — delete bounty",
|
||||
"/recover — recover deleted bounties",
|
||||
"",
|
||||
"📂 Categories:",
|
||||
"/category — list categories",
|
||||
"/category add — add category",
|
||||
"/category delete — delete category",
|
||||
"",
|
||||
"🔗 Tracking:",
|
||||
"/track — track bounty",
|
||||
"/untrack — stop tracking",
|
||||
|
||||
@@ -1,6 +1,20 @@
|
||||
"""Domain dataclasses for JIGAIDO bounty tracker."""
|
||||
|
||||
from dataclasses import dataclass
|
||||
from dataclasses import dataclass, field
|
||||
|
||||
|
||||
@dataclass
|
||||
class Category:
|
||||
"""A category for organizing bounties in a room.
|
||||
|
||||
Categories are per-room and support soft delete.
|
||||
The id (slug) must be lowercase alphabetic only (e.g., "bug", "feature").
|
||||
"""
|
||||
|
||||
id: str # slug: lowercase alphabetic only, e.g., "bug", "feature"
|
||||
name: str # display name: e.g., "Bug", "Feature"
|
||||
created_at: int
|
||||
deleted_at: int | None = None # soft delete
|
||||
|
||||
|
||||
@dataclass
|
||||
@@ -12,6 +26,8 @@ class Bounty:
|
||||
|
||||
The deleted_at field indicates soft-delete: None means not deleted,
|
||||
a value means deleted at that Unix timestamp.
|
||||
|
||||
The category_ids field lists category slugs associated with this bounty.
|
||||
"""
|
||||
|
||||
id: int
|
||||
@@ -22,6 +38,7 @@ class Bounty:
|
||||
created_by_user_id: int
|
||||
deleted_at: int | None = None
|
||||
created_by_username: str | None = None
|
||||
category_ids: list[str] = field(default_factory=list)
|
||||
|
||||
|
||||
@dataclass
|
||||
@@ -45,6 +62,7 @@ class RoomData:
|
||||
|
||||
The timezone field stores the room's timezone (e.g., "Asia/Jakarta"), default UTC+0.
|
||||
The admin_usernames field lists usernames who have admin privileges in this room.
|
||||
The categories field contains all categories for organizing bounties in this room.
|
||||
"""
|
||||
|
||||
room_id: int
|
||||
@@ -52,10 +70,13 @@ class RoomData:
|
||||
next_id: int
|
||||
timezone: str | None = None
|
||||
admin_usernames: list[str] | None = None
|
||||
categories: list[Category] = field(default_factory=list)
|
||||
|
||||
def __post_init__(self):
|
||||
if self.admin_usernames is None:
|
||||
self.admin_usernames = []
|
||||
if self.categories is None:
|
||||
self.categories = []
|
||||
|
||||
|
||||
@dataclass
|
||||
|
||||
230
core/services.py
230
core/services.py
@@ -3,7 +3,7 @@
|
||||
import time
|
||||
from typing import Optional
|
||||
|
||||
from core.models import Bounty, RoomData, TrackedBounty, TrackingData
|
||||
from core.models import Bounty, Category, RoomData, TrackedBounty, TrackingData
|
||||
from core.ports import RoomStorage, TrackingStorage
|
||||
|
||||
|
||||
@@ -289,6 +289,234 @@ class BountyService:
|
||||
results[bounty_id] = "deleted"
|
||||
return results
|
||||
|
||||
# --- Category Management ---
|
||||
|
||||
def add_category(
|
||||
self,
|
||||
room_id: int,
|
||||
slug: str,
|
||||
name: str,
|
||||
username: str | None,
|
||||
) -> Category:
|
||||
"""Create a new category. Admin only.
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
slug: Category ID (lowercase alphabetic, e.g., "bug")
|
||||
name: Display name (e.g., "Bug Report")
|
||||
username: Requesting admin's username
|
||||
|
||||
Returns:
|
||||
Created Category
|
||||
|
||||
Raises:
|
||||
PermissionError: If not admin
|
||||
ValueError: If slug already exists or invalid
|
||||
"""
|
||||
if not self.is_admin(room_id, username):
|
||||
raise PermissionError("Only admins can add categories.")
|
||||
|
||||
# Validate slug format (lowercase alphabetic only)
|
||||
if not slug or not slug.isalpha() or not slug.islower():
|
||||
raise ValueError(
|
||||
"Category slug must be lowercase alphabetic only (e.g., 'bug', 'feature')."
|
||||
)
|
||||
|
||||
room_data = self._storage.load(room_id)
|
||||
if room_data is None:
|
||||
room_data = RoomData(
|
||||
room_id=room_id, bounties=[], next_id=1, admin_usernames=[], categories=[]
|
||||
)
|
||||
|
||||
# Check for duplicate slug
|
||||
for cat in room_data.categories:
|
||||
if cat.id == slug and cat.deleted_at is None:
|
||||
raise ValueError(f"Category '{slug}' already exists.")
|
||||
|
||||
category = Category(
|
||||
id=slug,
|
||||
name=name,
|
||||
created_at=int(time.time()),
|
||||
deleted_at=None,
|
||||
)
|
||||
room_data.categories.append(category)
|
||||
self._storage.save(room_data)
|
||||
return category
|
||||
|
||||
def delete_category(
|
||||
self,
|
||||
room_id: int,
|
||||
slug: str,
|
||||
username: str | None,
|
||||
) -> bool:
|
||||
"""Soft delete a category. Admin only.
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
slug: Category slug to delete
|
||||
username: Requesting admin's username
|
||||
|
||||
Returns:
|
||||
True if deleted, False if not found
|
||||
"""
|
||||
if not self.is_admin(room_id, username):
|
||||
raise PermissionError("Only admins can delete categories.")
|
||||
|
||||
room_data = self._storage.load(room_id)
|
||||
if room_data is None:
|
||||
return False
|
||||
|
||||
for cat in room_data.categories:
|
||||
if cat.id == slug and cat.deleted_at is None:
|
||||
cat.deleted_at = int(time.time())
|
||||
self._storage.save(room_data)
|
||||
return True
|
||||
return False
|
||||
|
||||
def list_categories(self, room_id: int) -> list[Category]:
|
||||
"""List active categories (excludes soft-deleted).
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
|
||||
Returns:
|
||||
List of active categories
|
||||
"""
|
||||
room_data = self._storage.load(room_id)
|
||||
if room_data is None:
|
||||
return []
|
||||
return [c for c in room_data.categories if c.deleted_at is None]
|
||||
|
||||
def get_category(self, room_id: int, slug: str) -> Category | None:
|
||||
"""Get a category by slug (excludes soft-deleted).
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
slug: Category slug
|
||||
|
||||
Returns:
|
||||
Category or None if not found
|
||||
"""
|
||||
room_data = self._storage.load(room_id)
|
||||
if room_data is None:
|
||||
return None
|
||||
for cat in room_data.categories:
|
||||
if cat.id == slug and cat.deleted_at is None:
|
||||
return cat
|
||||
return None
|
||||
|
||||
def _validate_category_exists(self, room_id: int, slug: str) -> None:
|
||||
"""Validate that a category exists (and is not deleted). Raises ValueError if not found."""
|
||||
if not self.get_category(room_id, slug):
|
||||
raise ValueError(f"Category '{slug}' not found.")
|
||||
|
||||
def add_category_to_bounty(
|
||||
self,
|
||||
room_id: int,
|
||||
bounty_id: int,
|
||||
category_slug: str,
|
||||
username: str | None,
|
||||
) -> bool:
|
||||
"""Add category to a bounty. Admin only.
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
bounty_id: Bounty ID
|
||||
category_slug: Category slug to add
|
||||
username: Requesting admin's username
|
||||
|
||||
Returns:
|
||||
True if newly added, False if already exists
|
||||
|
||||
Raises:
|
||||
PermissionError: If not admin
|
||||
ValueError: If bounty or category not found
|
||||
"""
|
||||
if not self.is_admin(room_id, username):
|
||||
raise PermissionError("Only admins can manage bounty categories.")
|
||||
|
||||
bounty = self.get_bounty(room_id, bounty_id)
|
||||
if not bounty:
|
||||
raise ValueError("Bounty not found.")
|
||||
|
||||
self._validate_category_exists(room_id, category_slug)
|
||||
|
||||
if category_slug in bounty.category_ids:
|
||||
return False # Already exists
|
||||
|
||||
bounty.category_ids.append(category_slug)
|
||||
self._storage.update_bounty(room_id, bounty)
|
||||
return True
|
||||
|
||||
def remove_category_from_bounty(
|
||||
self,
|
||||
room_id: int,
|
||||
bounty_id: int,
|
||||
category_slug: str,
|
||||
username: str | None,
|
||||
) -> bool:
|
||||
"""Remove category from a bounty. Admin only.
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
bounty_id: Bounty ID
|
||||
category_slug: Category slug to remove
|
||||
username: Requesting admin's username
|
||||
|
||||
Returns:
|
||||
True if removed, False if not found
|
||||
"""
|
||||
if not self.is_admin(room_id, username):
|
||||
raise PermissionError("Only admins can manage bounty categories.")
|
||||
|
||||
bounty = self.get_bounty(room_id, bounty_id)
|
||||
if not bounty:
|
||||
raise ValueError("Bounty not found.")
|
||||
|
||||
if category_slug not in bounty.category_ids:
|
||||
return False
|
||||
|
||||
bounty.category_ids.remove(category_slug)
|
||||
self._storage.update_bounty(room_id, bounty)
|
||||
return True
|
||||
|
||||
def update_bounty_categories(
|
||||
self,
|
||||
room_id: int,
|
||||
bounty_id: int,
|
||||
category_slugs: list[str],
|
||||
username: str | None,
|
||||
) -> bool:
|
||||
"""Replace all categories on a bounty. Admin only.
|
||||
|
||||
Args:
|
||||
room_id: Room identifier
|
||||
bounty_id: Bounty ID
|
||||
category_slugs: New list of category slugs
|
||||
username: Requesting admin's username
|
||||
|
||||
Returns:
|
||||
True if updated
|
||||
|
||||
Raises:
|
||||
PermissionError: If not admin
|
||||
ValueError: If bounty or any category not found
|
||||
"""
|
||||
if not self.is_admin(room_id, username):
|
||||
raise PermissionError("Only admins can manage bounty categories.")
|
||||
|
||||
bounty = self.get_bounty(room_id, bounty_id)
|
||||
if not bounty:
|
||||
raise ValueError("Bounty not found.")
|
||||
|
||||
# Validate all categories exist
|
||||
for slug in category_slugs:
|
||||
self._validate_category_exists(room_id, slug)
|
||||
|
||||
bounty.category_ids = category_slugs
|
||||
self._storage.update_bounty(room_id, bounty)
|
||||
return True
|
||||
|
||||
|
||||
class TrackingService:
|
||||
"""Service for tracking bounty operations."""
|
||||
|
||||
Reference in New Issue
Block a user