Skip to content

chore: module path update#3

Merged
Snider merged 2 commits intomainfrom
dev
Mar 20, 2026
Merged

chore: module path update#3
Snider merged 2 commits intomainfrom
dev

Conversation

@Snider
Copy link
Copy Markdown
Contributor

@Snider Snider commented Mar 20, 2026

Summary

  • Updated go.mod module path to dappco.re/go/core

Test plan

  • go build clean
  • Pushed to Forge

🤖 Generated with Claude Code
Co-Authored-By: Virgil virgil@lethean.io

Summary by CodeRabbit

  • Chores
    • Updated the project module path to the new registry location.
  • Tests
    • All test suites updated to reference the core package from the new module path to ensure consistent compilation.
  • Generated Code
    • Emitters/fixtures now reference the core package via the updated module path so generated sources align with the change.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 20, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ab1a39bf-d38d-4e72-8b81-8fa940abba08

📥 Commits

Reviewing files that changed from the base of the PR and between 41c50da and 1728c29.

📒 Files selected for processing (23)
  • pkg/core/embed.go
  • tests/app_test.go
  • tests/array_test.go
  • tests/cli_test.go
  • tests/command_test.go
  • tests/config_test.go
  • tests/core_test.go
  • tests/data_test.go
  • tests/drive_test.go
  • tests/embed_test.go
  • tests/error_test.go
  • tests/fs_test.go
  • tests/i18n_test.go
  • tests/ipc_test.go
  • tests/lock_test.go
  • tests/log_test.go
  • tests/options_test.go
  • tests/runtime_test.go
  • tests/service_test.go
  • tests/string_test.go
  • tests/task_test.go
  • tests/testdata/scantest/sample.go
  • tests/utils_test.go
✅ Files skipped from review due to trivial changes (16)
  • tests/array_test.go
  • tests/log_test.go
  • tests/string_test.go
  • tests/core_test.go
  • tests/error_test.go
  • tests/command_test.go
  • tests/fs_test.go
  • tests/lock_test.go
  • tests/task_test.go
  • tests/drive_test.go
  • tests/i18n_test.go
  • tests/config_test.go
  • tests/testdata/scantest/sample.go
  • tests/service_test.go
  • tests/cli_test.go
  • tests/utils_test.go

📝 Walkthrough

Walkthrough

The module path in go.mod was changed from forge.lthn.ai/core/go to dappco.re/go/core. Corresponding import paths across generated code and all test files were updated to reference dappco.re/go/core/pkg/core.

Changes

Cohort / File(s) Summary
Module Configuration
go.mod
Module declaration changed from module forge.lthn.ai/core/go to module dappco.re/go/core. No other go.mod contents were modified.
Generated embed code
pkg/core/embed.go
GeneratePack updated to emit imports using dappco.re/go/core/pkg/core instead of forge.lthn.ai/core/go/pkg/core.
Test suite (dot-imports)
tests/*.go (e.g., tests/app_test.go, tests/array_test.go, tests/cli_test.go, tests/command_test.go, tests/config_test.go, tests/core_test.go, tests/data_test.go, tests/drive_test.go, tests/embed_test.go, tests/error_test.go, tests/fs_test.go, tests/i18n_test.go, tests/ipc_test.go, tests/lock_test.go, tests/log_test.go, tests/options_test.go, tests/runtime_test.go, tests/service_test.go, tests/string_test.go, tests/task_test.go, tests/utils_test.go)
All test files updated to dot-import dappco.re/go/core/pkg/core in place of forge.lthn.ai/core/go/pkg/core. Test logic and assertions unchanged.
Test fixtures / examples
tests/testdata/scantest/sample.go
Import path changed to dappco.re/go/core/pkg/core; code unchanged otherwise.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: module path update' directly reflects the main change: updating the Go module path from 'forge.lthn.ai/core/go' to 'dappco.re/go/core' across go.mod and all import statements.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.11.3)

Error: can't load config: unsupported version of the configuration: "" See https://golangci-lint.run/docs/product/migration-guide for migration instructions
The command is terminated due to an error: can't load config: unsupported version of the configuration: "" See https://golangci-lint.run/docs/product/migration-guide for migration instructions


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@go.mod`:
- Line 1: Update every Go import that still references the old module path
"forge.lthn.ai/core" to the new module path "dappco.re/go/core": search the repo
for the old import string and replace it in source, tests, generated code, and
docs. Also update the import-generation logic in the embed.go import-generation
routine so it emits "dappco.re/go/core" instead of the old path, and run `go
test`/`go build` to catch any remaining stale references.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 611d0a8f-70cc-4524-8a09-fc31a9feebc4

📥 Commits

Reviewing files that changed from the base of the PR and between cee07f0 and 41c50da.

📒 Files selected for processing (1)
  • go.mod

Comment thread go.mod
All .go imports, test fixtures, and embed.go code generation updated
to match the new module path.

Co-Authored-By: Virgil <virgil@lethean.io>
@Snider Snider merged commit d6dada1 into main Mar 20, 2026
1 check 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.

1 participant