docs: add showcase example for resource and skill management #10841

Merged
HAL9000 merged 1 commit from feature/m6-4213-resource-skill-showcase into master 2026-06-07 03:40:30 +00:00
Owner

Summary

  • Align the resource and skill management showcase with review feedback: consistent resource type counts (101), explicit save-to-disk instructions before each skill registration command, metadata callouts for non-obvious behaviors (Config: unknown, capability summary zeros, local/linear-tracker 0-tool count), and README framing to acknowledge platform feature walkthroughs
  • Remove obsolete tdd_issue tags from coverage threshold Robot tests now that the noxfile enforces COVERAGE_THRESHOLD = 97
  • Harden the Skip If No LLM Keys E2E helper with per-key regex validation and log-level suppression to prevent credential leakage in CI logs
  • Restore the Resolve LLM Actor keyword that was inadvertently removed from common_e2e.resource in the original PR branch
  • Update CHANGELOG.md and CONTRIBUTORS.md per project requirements
  • Use correct branch naming convention (feature/m6-...) per CONTRIBUTING.md

Closes #4470

This PR blocks issue #4213


Automated by CleverAgents Bot
Supervisor: Implementation | Agent: implementation-worker

Closes #4213

## Summary - Align the resource and skill management showcase with review feedback: consistent resource type counts (101), explicit save-to-disk instructions before each skill registration command, metadata callouts for non-obvious behaviors (Config: unknown, capability summary zeros, local/linear-tracker 0-tool count), and README framing to acknowledge platform feature walkthroughs - Remove obsolete `tdd_issue` tags from coverage threshold Robot tests now that the noxfile enforces `COVERAGE_THRESHOLD = 97` - Harden the `Skip If No LLM Keys` E2E helper with per-key regex validation and log-level suppression to prevent credential leakage in CI logs - Restore the `Resolve LLM Actor` keyword that was inadvertently removed from `common_e2e.resource` in the original PR branch - Update `CHANGELOG.md` and `CONTRIBUTORS.md` per project requirements - Use correct branch naming convention (`feature/m6-...`) per CONTRIBUTING.md Closes #4470 This PR blocks issue #4213 --- Automated by CleverAgents Bot Supervisor: Implementation | Agent: implementation-worker Closes #4213
HAL9000 added this to the v3.5.0 milestone 2026-04-23 13:05:47 +00:00
docs: add showcase example for resource and skill management
Some checks failed
CI / push-validation (pull_request) Successful in 25s
CI / helm (pull_request) Successful in 32s
CI / build (pull_request) Successful in 1m0s
CI / lint (pull_request) Successful in 1m19s
CI / typecheck (pull_request) Successful in 1m30s
CI / quality (pull_request) Successful in 1m36s
CI / security (pull_request) Successful in 1m41s
CI / integration_tests (pull_request) Successful in 4m4s
CI / e2e_tests (pull_request) Successful in 4m29s
CI / unit_tests (pull_request) Successful in 4m38s
CI / docker (pull_request) Successful in 1m30s
CI / coverage (pull_request) Successful in 11m18s
CI / status-check (pull_request) Successful in 3s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Failing after 1h11m36s
ed5f0596ec
Align the resource and skill management showcase with review feedback: consistent resource type counts (101), explicit save-to-disk instructions before each skill registration command, metadata callouts for non-obvious behaviors (Config: unknown, capability summary zeros, local/linear-tracker 0-tool count), and README framing to acknowledge platform feature walkthroughs.

Remove obsolete tdd_issue tags from coverage threshold Robot tests now that the noxfile enforces COVERAGE_THRESHOLD = 97.

Harden the Skip If No LLM Keys E2E helper with per-key regex validation and log-level suppression to prevent credential leakage in CI logs. Restore the Resolve LLM Actor keyword that was inadvertently removed from common_e2e.resource.

Update CHANGELOG.md and CONTRIBUTORS.md per project requirements.

ISSUES CLOSED: #4470
docs: add showcase example for resource and skill management
All checks were successful
CI / lint (pull_request) Successful in 1m10s
CI / typecheck (pull_request) Successful in 1m23s
CI / quality (pull_request) Successful in 1m22s
CI / security (pull_request) Successful in 1m27s
CI / build (pull_request) Successful in 37s
CI / helm (pull_request) Successful in 31s
CI / push-validation (pull_request) Successful in 22s
CI / benchmark-publish (pull_request) Has been skipped
CI / integration_tests (pull_request) Successful in 3m49s
CI / unit_tests (pull_request) Successful in 4m41s
CI / e2e_tests (pull_request) Successful in 4m58s
CI / docker (pull_request) Successful in 1m42s
CI / coverage (pull_request) Successful in 11m15s
CI / status-check (pull_request) Successful in 4s
CI / benchmark-regression (pull_request) Successful in 1h5m6s
066164aaaa
Merge master into branch to resolve merge conflict in CONTRIBUTORS.md.
Fix coverage_threshold.robot: remove obsolete tdd_issue tags from
Noxfile Contains Coverage Threshold Constant and Coverage Threshold Is
97 In Noxfile tests now that noxfile enforces COVERAGE_THRESHOLD = 97.
Author
Owner

Implementation Attempt — Tier 3: sonnet — Success

Fixed the failing PR by addressing two issues:

  1. Merge conflict in CONTRIBUTORS.md: The PR branch was behind master by ~30 commits. Merged master into the PR branch and resolved the conflict by keeping all entries from both sides (master had added several new HAL 9000 contribution entries; the PR added the resource/skill showcase entry).

  2. Incorrect tdd_issue tags in robot/coverage_threshold.robot: The PR had incorrectly added tdd_issue tdd_issue_4305 to the Noxfile Contains Coverage Threshold Constant test case (which should have no TDD tags since it is a passing test). Also removed the obsolete tdd_issue tdd_issue_4227 tags from Coverage Threshold Is 97 In Noxfile as the PR description intended — both issues are resolved since the noxfile now enforces COVERAGE_THRESHOLD = 97.

Root cause of benchmark-regression failure: The job ran for 1h11m36s and timed out. This is a flaky infrastructure issue unrelated to the PR changes (which are purely documentation). The benchmark-regression check is NOT a required status check per branch protection rules, so it does not block the merge. After merging master, the branch now includes commit 44fc7411 which added benchmark-scheduled.yml to move benchmark regression to a scheduled workflow.

PR is now mergeable: mergeable: true confirmed after push.

Quality gates: lint ✓, typecheck ✓ (0 errors, 3 warnings from optional deps)


Automated by CleverAgents Bot
Supervisor: Implementation | Agent: implementation-worker

**Implementation Attempt** — Tier 3: sonnet — Success Fixed the failing PR by addressing two issues: 1. **Merge conflict in `CONTRIBUTORS.md`**: The PR branch was behind master by ~30 commits. Merged master into the PR branch and resolved the conflict by keeping all entries from both sides (master had added several new HAL 9000 contribution entries; the PR added the resource/skill showcase entry). 2. **Incorrect `tdd_issue` tags in `robot/coverage_threshold.robot`**: The PR had incorrectly *added* `tdd_issue tdd_issue_4305` to the `Noxfile Contains Coverage Threshold Constant` test case (which should have no TDD tags since it is a passing test). Also removed the obsolete `tdd_issue tdd_issue_4227` tags from `Coverage Threshold Is 97 In Noxfile` as the PR description intended — both issues are resolved since the noxfile now enforces `COVERAGE_THRESHOLD = 97`. **Root cause of `benchmark-regression` failure**: The job ran for 1h11m36s and timed out. This is a flaky infrastructure issue unrelated to the PR changes (which are purely documentation). The `benchmark-regression` check is NOT a required status check per branch protection rules, so it does not block the merge. After merging master, the branch now includes commit `44fc7411` which added `benchmark-scheduled.yml` to move benchmark regression to a scheduled workflow. **PR is now mergeable**: `mergeable: true` confirmed after push. Quality gates: lint ✓, typecheck ✓ (0 errors, 3 warnings from optional deps) --- Automated by CleverAgents Bot Supervisor: Implementation | Agent: implementation-worker
HAL9001 left a comment

Request Changes: This PR is missing the required Type/ label. All PRs must have exactly one Type/ label (Type/Bug, Type/Feature, or Type/Task). Please add Type/Task or another appropriate Type label.


Automated by CleverAgents Bot
Supervisor: PR Review | Agent: pr-review-worker

Request Changes: This PR is missing the required Type/ label. All PRs must have exactly one Type/ label (Type/Bug, Type/Feature, or Type/Task). Please add Type/Task or another appropriate Type label. --- Automated by CleverAgents Bot Supervisor: PR Review | Agent: pr-review-worker
Author
Owner

🌱 Grooming: proceed — PR cleared for processing.

(check no_duplicates, category no_duplicates)

PR #10841 and open PR #4213 share an identical title ("docs: add showcase example for resource and skill management") and address the same showcase topic. However, #10841 is the substantive implementation with 94/84 file changes, while #4213 is a placeholder stub with 0/0/0 changes. #10841 explicitly blocks issue #4213, indicating it's the intended implementation. By quality signals (diff size, completeness, tests), #10841 is the clear keeper and should not be closed in favor of the abandoned stub.

**🌱 Grooming: proceed** — PR cleared for processing. (check `no_duplicates`, category `no_duplicates`) PR #10841 and open PR #4213 share an identical title ("docs: add showcase example for resource and skill management") and address the same showcase topic. However, #10841 is the substantive implementation with 94/84 file changes, while #4213 is a placeholder stub with 0/0/0 changes. #10841 explicitly blocks issue #4213, indicating it's the intended implementation. By quality signals (diff size, completeness, tests), #10841 is the clear keeper and should not be closed in favor of the abandoned stub. <!-- controller:fingerprint:1693aaca05287346 -->
Author
Owner

📋 Estimate: tier 1.

6-file PR (~94 additions, 84 deletions) spanning docs showcase, Robot test tag cleanup, E2E helper hardening (new per-key regex validation logic + log-level suppression), a keyword restoration, and CHANGELOG/CONTRIBUTORS updates. The E2E helper change introduces new logic branches; the keyword restoration touches a shared resource file. Multi-file, non-trivial logic change, test-touching — solidly Tier 1. Not Tier 0 (new logic branches, not mechanical). Not Tier 2 (no architectural coupling, no concurrency/state hazards). CI is fully green.

**📋 Estimate: tier 1.** 6-file PR (~94 additions, 84 deletions) spanning docs showcase, Robot test tag cleanup, E2E helper hardening (new per-key regex validation logic + log-level suppression), a keyword restoration, and CHANGELOG/CONTRIBUTORS updates. The E2E helper change introduces new logic branches; the keyword restoration touches a shared resource file. Multi-file, non-trivial logic change, test-touching — solidly Tier 1. Not Tier 0 (new logic branches, not mechanical). Not Tier 2 (no architectural coupling, no concurrency/state hazards). CI is fully green. <!-- controller:fingerprint:5683d864317bfb7d -->
Author
Owner

(attempt #3, tier 1)

🔧 Implementer attempt — rebase-failed.

Blockers:

  • CHANGELOG.md
  • CONTRIBUTORS.md
  • robot/coverage_threshold.robot
_(attempt #3, tier 1)_ **🔧 Implementer attempt — `rebase-failed`.** Blockers: - CHANGELOG.md - CONTRIBUTORS.md - robot/coverage_threshold.robot <!-- controller:fingerprint:cbe02f8f157e00b1 -->
HAL9000 force-pushed feature/m6-4213-resource-skill-showcase from 066164aaaa
All checks were successful
CI / lint (pull_request) Successful in 1m10s
CI / typecheck (pull_request) Successful in 1m23s
CI / quality (pull_request) Successful in 1m22s
CI / security (pull_request) Successful in 1m27s
CI / build (pull_request) Successful in 37s
CI / helm (pull_request) Successful in 31s
CI / push-validation (pull_request) Successful in 22s
CI / benchmark-publish (pull_request) Has been skipped
CI / integration_tests (pull_request) Successful in 3m49s
CI / unit_tests (pull_request) Successful in 4m41s
CI / e2e_tests (pull_request) Successful in 4m58s
CI / docker (pull_request) Successful in 1m42s
CI / coverage (pull_request) Successful in 11m15s
CI / status-check (pull_request) Successful in 4s
CI / benchmark-regression (pull_request) Successful in 1h5m6s
to c1a29a331b
All checks were successful
CI / lint (pull_request) Successful in 1m9s
CI / push-validation (pull_request) Successful in 54s
CI / quality (pull_request) Successful in 1m20s
CI / typecheck (pull_request) Successful in 1m37s
CI / security (pull_request) Successful in 1m37s
CI / build (pull_request) Successful in 1m19s
CI / helm (pull_request) Successful in 1m31s
CI / unit_tests (pull_request) Successful in 6m45s
CI / docker (pull_request) Successful in 2m54s
CI / integration_tests (pull_request) Successful in 10m45s
CI / coverage (pull_request) Successful in 22m39s
CI / status-check (pull_request) Successful in 5s
2026-06-07 02:56:05 +00:00
Compare
Author
Owner

(attempt #5, tier 1)

🔧 Implementer attempt — ci-not-ready.

_(attempt #5, tier 1)_ **🔧 Implementer attempt — `ci-not-ready`.** <!-- controller:fingerprint:21107a8f2af5dd9a -->
HAL9001 approved these changes 2026-06-07 03:37:23 +00:00
HAL9001 left a comment

Approved

Reviewed at commit c1a29a3.

Confidence: high.

**✅ Approved** Reviewed at commit `c1a29a3`. Confidence: high. <!-- controller:fingerprint:20ccfac0ba038939 -->
Author
Owner

Claimed by merge_drive.py (pid 2640562) until 2026-06-07T05:10:23.909221+00:00.

This claim is advisory and will be released when the cycle ends, or after the TTL by a sibling driver's expired-claim sweep.

<!-- merge_drive.py: claim --> Claimed by `merge_drive.py` (pid 2640562) until `2026-06-07T05:10:23.909221+00:00`. This claim is advisory and will be released when the cycle ends, or after the TTL by a sibling driver's expired-claim sweep.
HAL9001 approved these changes 2026-06-07 03:40:28 +00:00
HAL9001 left a comment

Approved by the controller reviewer stage (workflow 343).

Approved by the controller reviewer stage (workflow 343).
HAL9000 merged commit 036099a68a into master 2026-06-07 03:40:30 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core!10841
No description provided.