Skip to content

[codex] Remove persisted bounty summary column - #2798

Open
caydyan wants to merge 1 commit into
stakwork:masterfrom
caydyan:codex/remove-bounty-summary-column
Open

[codex] Remove persisted bounty summary column#2798
caydyan wants to merge 1 commit into
stakwork:masterfrom
caydyan:codex/remove-bounty-summary-column

Conversation

@caydyan

@caydyan caydyan commented Jun 13, 2026

Copy link
Copy Markdown

Summary

  • Stop persisting one_sentence_summary on Bounty and NewBounty by marking the field as non-GORM state while keeping the JSON field available for existing clients.
  • Drop the legacy one_sentence_summary column during database startup migrations when it still exists.
  • Derive the legacy response value from title, matching the issue note that both values were duplicates.
  • Remove migrated/seeded writes to the old column and add a schema test proving the summary field has no database column mapping.
  • Refresh the generated database mock with GetBountyByUnlockCode so the current test packages compile.

Fixes #590

Validation

  • go test ./db -run '^TestBountySummaryIsNotPersisted$' -count=1\n- go test ./db -run TestNonExistent -count=0\n- go test ./... -run TestNonExistent -count=0\n- git diff --check

@caydyan
caydyan marked this pull request as ready for review June 13, 2026 23:29
@caydyan

caydyan commented Jun 14, 2026

Copy link
Copy Markdown
Author

Current-head validation rerun on aab83a0:

  • go test ./db -run TestBountySummaryIsNotPersisted -count=1 -v passed, including both Bounty and NewBounty schema subtests.
  • go test ./... -run ^$ passed as a compile/no-test package sweep.
  • git diff --check origin/master...HEAD passed.

This focused test does not require the local Postgres test database, so it ran cleanly in this environment.

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.

Remove one_sentence_summary column from table

1 participant