cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

fix(v3.7.0): ResourceHandler return type #1444 #1486

Open
freemo wants to merge 7 commits from fix/1444-access-type into master
pull from: fix/1444-access-type
merge into: cleveragents:master
Conversation 63 Commits 7 Files Changed 18
+549 -61

7 Commits

Author SHA1 Message Date
controller-ci-rerun e854d47dfb chore: re-trigger CI [controller]
CI / load-versions (pull_request) Successful in 14s
Details
CI / push-validation (pull_request) Successful in 26s
Details
CI / build (pull_request) Successful in 32s
Details
CI / lint (pull_request) Successful in 58s
Details
CI / quality (pull_request) Successful in 56s
Details
CI / typecheck (pull_request) Failing after 1m10s
Details
CI / security (pull_request) Successful in 1m12s
Details
CI / helm (pull_request) Successful in 55s
Details
CI / unit_tests (pull_request) Successful in 5m13s
Details
CI / docker (pull_request) Has been skipped
Details
CI / coverage (pull_request) Has been skipped
Details
CI / integration_tests (pull_request) Successful in 8m50s
Details
CI / status-check (pull_request) Failing after 3s
Details
2026-06-15 07:08:26 -04:00
controller-ci-rerun b3d0968340 chore: re-trigger CI [controller] 2026-06-15 07:08:26 -04:00
HAL9000 8d2934c0c0 fix(ci): apply ruff format and suppress vulture containment_path 
Reformat features/steps/resource_handler_project_access_1444_steps.py
and src/cleveragents/resource/handlers/_base.py to satisfy ruff format
check (complex slice spacing, collapsing single-f-string assert parens).

Add containment_path to vulture_whitelist.py — the parameter is
required by the ResourceHandler.project_access interface (spec
§21691-21698) but is intentionally unused in stub implementations
that raise NotImplementedError.
 2026-06-15 07:08:26 -04:00
controller-ci-rerun 853c9ea7ef chore: re-trigger CI [controller] 2026-06-15 07:08:26 -04:00
HAL9000 3b7fdb0897 fix(resource): restore database checkpoint connection reuse and update changelog 
Database handler now reuses checkpoint connections to preserve SAVEPOINT semantics.

Restore _get_checkpoint_conn method in DatabaseResourceHandler that was
accidentally removed. This method is essential for SAVEPOINT semantics —
writes must share the checkpoint's open connection so rollback_to() can
restore them during a checkpoint transaction.

- Add CHANGELOG.md entry under [Unreleased] for #1444 fix
- Update CONTRIBUTORS.md with ResourceHandler fix attribution

ISSUES CLOSED: #1444
Epic-Reference: #398
 2026-06-15 07:08:26 -04:00
HAL9000 fa0065593b fix(resource): implement AccessProjection return type on ResourceHandler.project_access 
Implements the spec-required AccessProjection type with fields access_path,
protocol, crosses_sandbox, and read_richness per docs/specification.md lines 
25100-25114. Updates ResourceHandler.project_access to accept the spec-required
parameters (binding_resource, target_resource, containment_path, sandbox) and
return AccessProjection | None per spec lines 21691-21698.

Separates access-control logic into a new check_access method returning 
AccessCheckResult, resolving the semantic confusion between authorization
(check_access) and routing/reachability (project_access).

Updates all concrete handler implementations and test step definitions to use
the new signatures. Adds TDD Behave scenarios tagged @tdd_issue @tdd_issue_1444
verifying the spec-required fields and method signatures.

ISSUES CLOSED: #1444
Epic-Reference: #398
 2026-06-15 06:57:28 -04:00
freemo 41d9d4a88b fix(v3.7.0): resolve issue #1444 
Refactored ResourceHandler return types: split singular AccessResult into 
AccessProjection (routing/reachability) and AccessCheckResult (authorization).

ISSUES CLOSED: #1444
Epic-Reference: #398
 2026-06-15 06:57:28 -04:00