datatug / backend / 41819250535
100%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 410
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 99.995%. Remained the same
41819250535

push

github

web-flow 
test(architecture): forbid dalgo2* adapter module imports (#151)

Adds `internal/architecture/no_adapter_imports_test.go` (test-only package, no production code) and a new `AGENTS.md` rule: backend code must not import a separate DALgo adapter module, `github.com/dal-go/dalgo2<anything>`. Imports from the DALgo core module, including `dalgo/adapters/dalgo2memory` and `dalgo2fs`, are fine.

The check is a plain text search over every `.go` file (tests included, whatever build tags). The 21 files that import an adapter on origin/main 891ce1c (3 non-test: `facade4datatug/shipping_original_log.go`, `googlecloudlogs4datatug/reader.go`, `googlecloudlogs4datatug/reader_result.go`; 18 tests, dalgo2firestore and dalgo2logs) sit on `adapterImportDebt`, marked as debt to remove. The test fails when:

- a file not on the list imports a `dalgo2*` module;
- a listed file no longer imports one (delete the entry: the list only shrinks);
- go.mod's direct (non-`// indirect`) `dalgo2*` requirements differ from the modules the listed files import.

Positive controls cover each failure kind plus the core-module negatives. Runs under plain `go test ./...` in strongo/cicd's workflow (backend-ci.yml).

Planted-violation results:
- new `models4datatug/zz_planted.go` importing dalgo2firestore → fails, naming the file;
- `infrastructureroots/persistence_firestore_integration_test.go` with its import swapped to core → fails, "delete its adapterImportDebt entry";
- go.mod `dalgo2sql` turned direct → fails, "requires … directly but no listed file imports it".
All reverted; the test passes on the branch.

Founder, 2026-11-04: "Can we also add CIcheck we do not import any dalgo2* adapters? Somethings simple like maybe regex or just text search?" / "We are OK to import from core but not from other modules/repos" / "Core has memory and file system adapters I believe"

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Source commits:

- cfa8b1f81e21 test(architecture): f... (continued)

54973 of 54976 relevant lines covered (99.99%)

1.0 hits per line

Jobs
ID Job ID Ran Files Coverage
1 41819250535.1 410
99.99
GitHub Action Run
Source Files on build 41819250535