fix: migrate module paths to dappco.re

[Snider]

Module path migration from forge.lthn.ai to dappco.re

Summary by CodeRabbit

• New Features

  • Added DeleteMany() method to remove multiple cache entries in a single operation.

• Bug Fixes

  • Improved safety with validation for invalid cache states.
  • Enhanced time-to-live (TTL) validation to reject invalid durations.

• Documentation

  • Published API contract reference table for all exported methods.
  • Added architecture and security analysis documentation.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The cache implementation has been refactored to replace filesystem-specific path handling with medium-agnostic helpers whilst adding configuration/readiness guards. Error handling and serialization now leverage core utilities. TTL validation was introduced, and a bulk deletion API (DeleteMany) was added. Tests were comprehensively expanded, and documentation covering API contracts, architecture, and security analysis was added. Module dependencies were updated to target a newer core version.

Changes

Cohort / File(s)	Summary
Core Implementation Refactoring cache.go	Replaced filepath and os operations with core.JoinPath, core.CleanPath, and core.Concat for path handling. Introduced ensureConfigured and ensureReady guards to fail early when cache is misconfigured. Updated serialization to use core.JSONMarshal and core.JSONUnmarshal, with TTL validation rejecting negative values. Changed error checking from os.ErrNotExist to fs.ErrNotExist. Added DeleteMany(keys...) API for bulk deletion. Updated key helper functions to use medium-agnostic path construction.
Test Suite Expansion cache_test.go	Replaced single monolithic test with focused test suite covering constructor TTL validation, path handling with traversal rejection, get/set/expiry behaviour, receiver safety for nil and zero-value caches, bulk deletion semantics, and key helper validation. Introduced newTestCache and readEntry helpers for test infrastructure.
Documentation docs/api-contract.md, docs/architecture.md, docs/security-attack-vector-mapping.md	Added new API contract reference documenting all exported symbols and test coverage. Updated architecture documentation for DeleteMany behaviour, path-traversal validation logic, and GitHub key helper implementations. Added comprehensive security attack vector mapping analysing input sources, validation present, and residual risks across the public API.
Dependency Management go.mod	Updated to directly require dappco.re/go/core v0.8.0-alpha.1 and removed direct requirement for dappco.re/go/core/log v0.1.0, retargeting the indirect go-log dependency accordingly.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 13.04% 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 'fix: migrate module paths to dappco.re' accurately describes the main change in the changeset—updating module paths and dependencies from forge.lthn.ai to dappco.re. However, the changeset encompasses significantly more substantial work, including core cache functionality improvements, new test coverage, security documentation, and API contract documentation.

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

---

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