fix(tag-rules): resolve joined table through the FK field's possible-value source #10

2026-10-13T00:47:44Z

KofTwentyTwo commented

2026-10-13 00:47:44 +00:00

(Migrated from github.com)

Problem

`TagRuleEngine` used a placeholder prefix as both the FK field name and the table name: `{blog.slug}` read `blogId` then GOT table `blog`. Apps with prefixed tables (kof22 `cms_blog`) silently lost those purge paths.

Fix

The joined table is resolved from `Id`'s possible-value source (table name for TABLE-type, or the PVS name when it names a table); fallback to the bare prefix keeps existing behaviour.
A missing joined table no longer skips silently: warn once per table per engine; a missing FK field/value is skipped without throwing.
README section on placeholders and CHANGELOG 0.1.1 entry.

Tests

Prefixed table via PVS, PVS-named-as-table, bare-prefix fallback, missing FK field, missing/dangling table. `CI=true ./mvnw verify`: 77 tests pass, checkstyle and coverage checks met.

🤖 Generated with Claude Code

Replaces #9. The hotfix/* branch name triggered the hotfix publish workflow instead of test CI. This is the same commit on a fix/* branch.

## Problem \`TagRuleEngine\` used a placeholder prefix as both the FK field name and the table name: \`{blog.slug}\` read \`blogId\` then GOT table \`blog\`. Apps with prefixed tables (kof22 \`cms_blog\`) silently lost those purge paths. ## Fix - The joined table is resolved from \`<prefix>Id\`'s possible-value source (table name for TABLE-type, or the PVS name when it names a table); fallback to the bare prefix keeps existing behaviour. - A missing joined table no longer skips silently: warn once per table per engine; a missing FK field/value is skipped without throwing. - README section on placeholders and CHANGELOG 0.1.1 entry. ## Tests Prefixed table via PVS, PVS-named-as-table, bare-prefix fallback, missing FK field, missing/dangling table. \`CI=true ./mvnw verify\`: 77 tests pass, checkstyle and coverage checks met. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Replaces #9. The `hotfix/*` branch name triggered the hotfix publish workflow instead of test CI. This is the same commit on a `fix/*` branch.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: QRun-IO/qbit-cdn#10