Remove slug field from Tag model (#1789) ## Summary This PR removes the `slug` field from the Tag model and all related slug generation logic. Tags are now identified and referenced by their name instead of a generated slug, simplifying the data model and reducing complexity. ## Related issues N/A ## Changes these areas - [x] Internal architecture - [x] Snapshot data layout on disk ## Details ### What changed 1. **Model changes**: Removed the `slug` field from the Tag model, including the `_generate_unique_slug()` method and slug generation logic in the `save()` method 2. **Database migration**: Added migration `0034_remove_tag_slug` to drop the slug column 3. **API updates**: Removed `slug` from all API schemas (TagSchema, TagSearchCardSchema, TagUpdateResponseSchema) and responses 4. **Tag lookup**: Updated `get_tag_by_ref()` to use URL-decoded tag names instead of slugs for lookups 5. **Tag filtering**: Simplified `get_matching_tags()` to only filter by name instead of both name and slug 6. **Export filenames**: Changed tag export filenames to use `quote(tag.name)` instead of `tag.slug` 7. **Admin interface**: Removed slug from TagAdmin search fields, readonly fields, and fieldsets 8. **Templates**: Removed slug display from tag cards and similar tags UI 9. **Tests**: Updated test expectations and removed slug assertions; updated export filename checks to use `quote(tag.name)` ### Why This simplifies the Tag model by removing the derived slug field. Tags can be uniquely identified by their name, and URL encoding handles special characters in filenames and URLs. This reduces database complexity and eliminates the need for slug generation and uniqueness logic. ## Test Plan Existing tests have been updated to verify the new behavior: - `test_tag_rename_api_updates_name` verifies tag renaming works without slug - `test_tag_snapshots_export_returns_jsonl` and `test_tag_urls_export_returns_plain_text_urls` verify export filenames use encoded tag names - `test_tag_table_has_required_columns` verifies the database schema no longer includes slug All related tests pass with the updated assertions. https://claude.ai/code/session_014KmEXoA64Ayp2t8BW2xfVP ---

--- ## Summary by cubic Removed the stored `slug` from Tag and moved to name-based tags. Added a derived `Tag.slug` via `django.utils.text.slugify` for clean export filenames and an admin download fallback; public APIs no longer include slugs and lookups resolve by URL-decoded exact name. - **Refactors** - Replaced stored slug with a derived `Tag.slug` property; removed slug generation/save logic. - Public API schemas and autocomplete drop `slug`; matching/filtering uses `name` only. - `get_tag_by_ref` resolves by URL-decoded `name` (case-insensitive exact match). - Export endpoints set filenames using `tag.slug`; admin tag cards expose `data-slug`, and the client uses it as a fallback filename. Removed slug from admin search fields/fieldsets and UI displays. - **Migration** - Run database migrations. - Update any consumers expecting `slug` in Tag API/admin; use the tag `name` for references (URL-encode names in links). Rely on server-provided filenames, with the built-in client fallback using `tag.slug` where needed. ^{Written for commit 7c3a3e0dba0e1a845289926a834255a54420d148.
Summary will update on new commits.}

Sunbelt Computer Software

PL/B Language Development and Support

Uh oh!