Skip to content

Conversation

@aaronstevenwhite
Copy link
Contributor

Summary

This PR fixes TypeScript compilation errors in the scripts/ directory and includes all scripts in the compilation by updating tsconfig.json.

Changes:

  • tsconfig.json: Changed include from scripts/db/**/* to scripts/**/* to compile all scripts
  • scripts/sync-did.ts: Rewrote to use EprintsRepository with proper type conversion from Eprint to StoredEprint
  • scripts/cleanup-pds-records.ts: Fixed to use AtpAgent properly with authentication for delete operations
  • scripts/bluesky-post.ts: Fixed undefined handling in regex destructuring and session access
  • scripts/alpha-admin.ts: Removed unused import and fixed null/undefined type mismatches
  • scripts/reindex-all-eprints.ts: Removed unused resolveElasticsearchIndex function
  • scripts/admin-index-record.ts: Removed (was using non-existent imports)
  • scripts/db/recreate-elasticsearch-index.ts: New script to recreate Elasticsearch index with correct field_nodes nested mapping

Related Issues

Related to field_nodes nested type fix for Elasticsearch search filtering.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Refactoring (no functional changes)
  • Documentation update

How Has This Been Tested?

  • TypeScript compilation passes with no errors
  • All 2616 unit tests pass

Checklist

General

  • I have performed a self-review of my code
  • Code follows style guide (npm run lint passes)
  • Tests added/updated for changes
  • All new and existing tests pass (npm test)
  • Documentation updated (if applicable)

ATProto Compliance (required for data flow changes)

  • N/A - No data flow changes, only script fixes

Breaking Changes

  • N/A — no breaking changes

@aaronstevenwhite aaronstevenwhite merged commit 9e51e7d into main Jan 26, 2026
5 checks passed
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.

2 participants