refactor: replace tokens_module with sessions_module in export#675
Merged
pyramation merged 2 commits intomainfrom Jan 28, 2026
Merged
refactor: replace tokens_module with sessions_module in export#675pyramation merged 2 commits intomainfrom
pyramation merged 2 commits intomainfrom
Conversation
- Replace tokens_module with sessions_module in tableOrder (export-migrations.ts) - Replace tokens_module config with sessions_module config (export-meta.ts) - Remove tokens_table_id from rls_module config - Remove tokens_table_id from user_auth_module config - Update crypto_auth_module to use sessions_table_id and session_credentials_table_id
Contributor
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…dule and user_auth_module configs - Add sessions_table_id and session_credentials_table_id to rls_module config - Add sessions_table_id and session_credentials_table_id to user_auth_module config - Add missing audits_table_id, audits_table_name, verify_password_function, check_password_function to user_auth_module
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
refactor: replace tokens_module with sessions_module in export
Summary
Updates the export functionality to use the new
sessions_moduleinstead of the deprecatedtokens_module. This is part of the session-centric auth migration where all authentication tokens now live insession_credentialstable instead ofapi_tokens.Changes:
tokens_modulewithsessions_moduleintableOrderarray (export-migrations.ts)tokens_moduleconfig withsessions_moduleconfig with new field structure (export-meta.ts)tokens_table_idwithsessions_table_idandsession_credentials_table_idinrls_moduleconfigtokens_table_idwithsessions_table_idandsession_credentials_table_idinuser_auth_moduleconfig (also added missingaudits_table_id,audits_table_name,verify_password_function,check_password_function)tokens_table_idwithsessions_table_idandsession_credentials_table_idincrypto_auth_moduleconfigReview & Testing Checklist for Human
rls_module,user_auth_module,crypto_auth_module, andsessions_moduleconfigs against the actual table definitions in constructive-db to ensure they match exactlygenerate:constructiveor similar export command after dependent PRs are merged to verify the export works end-to-endNotes
packages/csv-to-pg/__tests__/export.test.tsstill usestokens_moduleas an example table name for parser tests - this is just a test fixture and doesn't affect functionality__fixtures__/output/schemas/services_public.tsfile will need to be regenerated after the dependent PRs are mergedLink to Devin run: https://app.devin.ai/sessions/d30b2724020f43f6b94865bbd5cf71c6
Requested by: Dan Lynch (@pyramation)