Skip to content

feat: Implement topic aliases + fix bugs - #7

Merged
Bertie690 merged 5 commits into
mainfrom
aliases
May 8, 2026
Merged

feat: Implement topic aliases + fix bugs#7
Bertie690 merged 5 commits into
mainfrom
aliases

Conversation

@Bertie690

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds first-class help-topic aliases backed by a new topic_aliases table, integrates alias resolution into topic lookup/autocomplete, and introduces new Discord bot commands for managing aliases.

Changes:

  • Add DB schema + store/db APIs for topic aliases (list/add/delete) and wire migrations to create/drop topic_aliases.
  • Update /help and autocomplete to optionally include aliases, and add /add-alias + /remove-alias commands.
  • Refactor a few DB access patterns and tighten some command UX/validation helpers.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
db/store.go Adds alias data model + CRUD/query methods; minor refactors to topic retrieval/deletion.
db/migrations/00002_add_aliases.up.sql Creates the topic_aliases table.
db/migrations/00002_add_aliases.down.sql Drops the topic_aliases table.
db/migrations.go Embeds/applies the new migration and improves migration logging metadata.
db/db.go Exposes alias APIs and updates topic retrieval to resolve aliases.
commands/utils.go Adds shared topic/alias validation and optional alias inclusion in topic lists.
commands/remove-alias.go New /remove-alias command.
commands/help.go Enables alias-inclusive autocomplete for /help.
commands/get-topics.go Adds include-aliases option and passes it through to topic listing output.
commands/delete-topic.go Updates messaging and autocomplete behavior around deletions.
commands/constants.go Adds a constant for autocomplete choice display length.
commands/autocomplete.go Adds alias-inclusive autocomplete + alias-only autocomplete helper.
commands/all-commands.go Registers the new alias commands.
commands/add-help.go Adjusts add-help UX to resolve aliases and refactors timestamp encoding helpers.
commands/add-alias.go New /add-alias command.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread db/store.go
Comment thread db/store.go Outdated
Comment thread db/store.go
Comment thread db/migrations/00002_add_aliases.up.sql
Comment thread commands/utils.go Outdated
Comment thread commands/get-topics.go Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Comment thread db/migrations/00002_add_aliases.up.sql Outdated
Comment thread commands/utils.go Outdated
Comment thread commands/autocomplete.go Outdated
Comment thread commands/autocomplete.go
Comment thread commands/add-help.go Outdated
@Bertie690
Bertie690 merged commit 7bce39b into main May 8, 2026
2 checks passed
@Bertie690
Bertie690 deleted the aliases branch May 8, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants