Confirm cogs/gameplay_legacy.py is excluded from production #42

New Issue

cal · 2026-02-20T06:53:01Z

cal commented

2026-02-20 06:53:01 +00:00

~2500+ lines, not listed in paperdynasty.py's COGS list, but exists in Docker image. Still imports live production dependencies. Risks accidental activation if cog list were ever auto-discovered.

~2500+ lines, not listed in `paperdynasty.py`'s `COGS` list, but exists in Docker image. Still imports live production dependencies. Risks accidental activation if cog list were ever auto-discovered.

cal added the

tech-debt

label 2026-02-20 06:53:01 +00:00

cal added the

ai-working

label 2026-03-05 06:01:23 +00:00

cal referenced this issue from a commit

2026-03-05 06:04:33 +00:00

fix: explicitly exclude cogs/gameplay_legacy.py from Docker image (#42)

cal referenced this issue

2026-03-05 06:04:52 +00:00

fix: explicitly exclude cogs/gameplay_legacy.py from Docker image (#42) #52

cal removed the

ai-working

label 2026-03-05 06:05:10 +00:00

cal commented

2026-03-05 06:05:14 +00:00

PR opened: #52

The .dockerignore already had a *_legacy.py wildcard (added in an earlier catchup commit) which should exclude this file, but it was implicit. Added an explicit cogs/gameplay_legacy.py entry to make the exclusion unambiguous and self-documenting.

Note: the file still exists in the repo and imports live production dependencies. Deleting it entirely would be a more complete fix — flagged in the PR description for review.

PR opened: https://git.manticorum.com/cal/paper-dynasty-discord/pulls/52 The `.dockerignore` already had a `*_legacy.py` wildcard (added in an earlier catchup commit) which should exclude this file, but it was implicit. Added an explicit `cogs/gameplay_legacy.py` entry to make the exclusion unambiguous and self-documenting. Note: the file still exists in the repo and imports live production dependencies. Deleting it entirely would be a more complete fix — flagged in the PR description for review.

cal added the

ai-pr-opened

label 2026-03-05 06:05:25 +00:00

cal referenced this issue from a commit

2026-03-10 14:11:57 +00:00

Merge pull request 'fix: explicitly exclude cogs/gameplay_legacy.py from Docker image (#42)' (#52) from ai/paper-dynasty-discord-42 into next-release

cal closed this issue

2026-03-10 18:26:33 +00:00

cal referenced this issue

2026-03-20 13:30:51 +00:00

release: Card Evolution Phase 1c + run-decision algorithm #103

Sign in to join this conversation.