Cog-Creators / Cog-Creators/Red-DiscordBot

Migrate Alias's config from a list of entries to a `name->alias_data` mapping

Open
#5,242 0 comments 0 reactions 0 assignees View on GitHub
Category: Cogs - Alias Status: In Progress Type: Enhancement Type: Optimisation
Dominant language
Python
Stars
5.7k
Forks
2.5k
Avg merge
6d 16h
Merged PRs (30d)
1

Description

### What component of Red (cog, command, API) would you like to see improvements on?

Alias

### Describe the enhancement you're suggesting.

The current data structure seems inefficient. This isn't much of an issue when calling an alias since we maintain a cache in the cog but it is not ideal for edits or deletions which require finding the alias in the entries list. Using a custom group instead would be preferred.

Low-priority issue but would IMO be nice to change this at some point.

### Anything else?

_No response_

Contributor guide

Open the contributing guide

Research direction

Start by locating the Alias cog's configuration entries and its cache, then trace how edits and deletions find aliases in the list. Review the existing custom-group support and determine the migration path for the stored data. Done means Alias configuration uses a name-to-alias_data mapping while calling, editing, and deleting aliases continue to work.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.