Skip to content

Feat: Codex migration #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
nipsysdev wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Feat: Codex migration #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a864688
feat(codexMigration): initial push
nipsysdev Nov 20, 2025
2e1cfd3
chore: remove dead code
nipsysdev Nov 20, 2025
57e03b5
feat: display peer id and listen addresses on start
nipsysdev Nov 20, 2025
d9701a4
test: update tests
nipsysdev Nov 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 37 additions & 0 deletions .env.example
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
# Server Configuration
SERVER_PORT=8000
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The SERVER_PORT environment variable on line 2 is not used in the new configuration implementation (LocalitySrvConfig::from_env()). It should be removed from the example file as the application no longer runs an HTTP server.

Suggested change
SERVER_PORT=8000

Copilot uses AI. Check for mistakes.
ASSETS_DIR=./assets

# Command-line Tool Paths
PMTILES_CMD=pmtiles
BZIP2_CMD=bzip2
FIND_CMD=find

# Database Configuration
LOCALITYSRV_DB_PATH=./assets/whosonfirst-data-admin-latest.db
LOCALITYSRV_CID_DB_PATH=./assets/locality-cid-mappings.db
LOCALITYSRV_LOCALITIES_DIR=./assets/localities
WHOSEONFIRST_DB_URL=https://data.geocode.earth/wof/dist/sqlite/whosonfirst-data-admin-latest.db.bz2
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in environment variable name: "WHOSEONFIRST_DB_URL" should be "WHOSONFIRST_DB_URL" (missing the second 'O' in "WHOSONFIRST").

Suggested change
WHOSEONFIRST_DB_URL=https://data.geocode.earth/wof/dist/sqlite/whosonfirst-data-admin-latest.db.bz2
WHOSONFIRST_DB_URL=https://data.geocode.earth/wof/dist/sqlite/whosonfirst-data-admin-latest.db.bz2

Copilot uses AI. Check for mistakes.

# Protomaps Configuration
PROTOMAPS_BUILDS_URL=https://build-metadata.protomaps.dev/builds.json

# Local Planet PMTiles Configuration
# Set this to the path of a local planet.pmtiles file to use it instead of downloading from remote
# Example: PLANET_PMTILES_PATH=/path/to/planet.pmtiles
# If not set or empty, the system will fetch the latest planet pmtiles from the remote URL
PLANET_PMTILES_PATH=

# Target Countries (comma-separated, empty or ALL for all countries)
TARGET_COUNTRIES=AE,AF

# Codex Node Configuration
CODEX_DATA_DIR=./.codex-data
CODEX_STORAGE_QUOTA_GB=100
CODEX_DISCOVERY_PORT=8090
CODEX_LISTEN_ADDRS=/ip4/0.0.0.0/tcp/8090,/ip4/127.0.0.1/tcp/8090
LOG_LEVEL=info

# Performance Settings
MAX_CONCURRENT_EXTRACTIONS=10
DB_CONNECTION_POOL_SIZE=10
Comment on lines +16 to +37
Copy link

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These environment variables in the example file (PROTOMAPS_BUILDS_URL and DB_CONNECTION_POOL_SIZE) are no longer used in the codebase. They should be removed from the example configuration file to avoid confusion.

Copilot uses AI. Check for mistakes.
5 changes: 4 additions & 1 deletion .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,7 @@
*.bz2
*.db
*.pmtiles
.kilocode
.kilocode
.env
.codex-data
.codex-test-data
Loading