Skip to content

docs: use placeholder account, entity and service names in examples and tests - #145

Merged
piekstra merged 1 commit into
mainfrom
piekstra/generic-examples
Oct 13, 2026
Merged

piekstra merged 1 commit into
mainfrom
piekstra/generic-examples

Conversation

@piekstra

Copy link
Copy Markdown
Member

Examples and test fixtures now use placeholder values: account ID 1234567, entity ID 123456789, and checkout-service as the service name. Entity GUIDs in examples are re-encoded from the placeholder values, so they still decode to the same shape.

Tests: make test passes.

@monit-reviewer monit-reviewer left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Automated PR Review

Reviewed commit: 42cb74f42f51
Profile: claude-monit-reviewer - Posting as: monit-reviewer

Summary

Reviewer Findings
go:implementation-tests 0
policies:conventions 0

Reviewer Coverage

  • go:implementation-tests — complete (broad); inspected 5 assigned files (6 inspected across reviewers): api/resolve_test.go, internal/cmd/dashboards/dashboards.go, internal/cmd/deployments/deployments.go, internal/cmd/logs/logs.go, internal/deeplink/deeplink_test.go; skipped: none; constraints: none
  • policies:conventions — complete (broad); inspected 4 assigned files (6 inspected across reviewers): README.md, internal/cmd/dashboards/dashboards.go, internal/cmd/deployments/deployments.go, internal/cmd/logs/logs.go; skipped: none; constraints: No local or sibling copies of open-cli-collective/cli-common or .github docs were present in the workbench, so comparison relied on in-repo conventions only.
Inspected files (6)
  • README.md
  • api/resolve_test.go
  • internal/cmd/dashboards/dashboards.go
  • internal/cmd/deployments/deployments.go
  • internal/cmd/logs/logs.go
  • internal/deeplink/deeplink_test.go

0 PR discussion threads considered. 0 summarized; 0 resolved.


Completed in 1m 59s | ~$0.82 (est.) | claude-sonnet-5 | cr 0.10.319
Field Value
Model claude-sonnet-5
Reviewers go:implementation-tests, policies:conventions
Engine claude_cli · claude-sonnet-5
Reviewed by cr · monit-reviewer
Duration 1m 59s wall · 2m 07s compute
Cost ~$0.82 (est.)
Pricing basis anthropic-public-2026-09-23
Tokens 30 in / 6.6k out

Per-workstream usage

  • orchestrator-selection — claude-sonnet-5
    • In: 6
    • Out: 3.3k
    • Cache read: 90.3k
    • Cache create: 43.3k
    • Cost: ~$0.22 (est.)
    • Duration: 35s
  • go:implementation-tests — claude-sonnet-5
    • In: 8
    • Out: 965
    • Cache read: 158.0k
    • Cache create: 42.1k
    • Cost: ~$0.21 (est.)
    • Duration: 24s
  • policies:conventions — claude-sonnet-5
    • In: 12
    • Out: 1.9k
    • Cache read: 268.5k
    • Cache create: 40.2k
    • Cost: ~$0.23 (est.)
    • Duration: 57s
  • orchestrator-rollup — claude-sonnet-5
    • In: 4
    • Out: 376
    • Cache read: 84.8k
    • Cache create: 32.9k
    • Cost: ~$0.15 (est.)
    • Duration: 10s

@piekstra
piekstra marked this pull request as ready for review October 13, 2026 13:19
@piekstra
piekstra merged commit 9c70d3f into main Oct 13, 2026
10 checks passed
@piekstra
piekstra deleted the piekstra/generic-examples branch October 13, 2026 13:20
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.

2 participants