Skip to content

feat: Add weekly-report.sh scoped to core repos - #59

Open
rubambiza wants to merge 2 commits into
rossoctl:mainfrom
rubambiza:feat/weekly-report-wrapper
Open

feat: Add weekly-report.sh scoped to core repos#59
rubambiza wants to merge 2 commits into
rossoctl:mainfrom
rubambiza:feat/weekly-report-wrapper

Conversation

@rubambiza

Copy link
Copy Markdown
Contributor

Summary

Adds scripts/weekly-report.sh, a thin wrapper that scopes the weekly org
report to the curated core-repo allowlist instead of discovering every repo
in the org.

The wrapper resolves the org identity and the core-repo list through the
shared library (load_org_profile + get_core_repos), then invokes the
Python report generator with --repos set to exactly those repos. When the
generator is called without --repos (the previous default), org-wide
discovery is unchanged, so this is additive.

Changes

  • scripts/weekly-report.sh — wrapper: resolves org + core allowlist, builds
    report.py --org <org> --repos <core repos> [--since/--until/--output/--json-output],
    fails loud when the allowlist is empty or the generator is missing.
    Supports --profile and --org overrides.
  • tests/test-weekly-report.sh — hermetic test (fixture allowlist via
    CORE_REPOS_FILE, stub generator via REPORT_PY): asserts arg-building,
    --since/--until pass-through, and fail-loud on a missing generator. No
    network or real config touched.
  • .github/workflows/tests.yml — wires the new test into the suite runner.

The companion --repos support in the generator lives in
rossoctl/agent-skills (separate PR).

Fixes #57

Assisted-By: Claude Code

Assisted-By: Claude Code (Anthropic AI) <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
Assisted-By: Claude Code (Anthropic AI) <noreply@anthropic.com>

Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
@rubambiza rubambiza added the ready-for-ai-review Request automated AI code review from clawgenti label Aug 18, 2026

@clawgenti clawgenti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Adds scripts/weekly-report.sh, a thin wrapper that resolves the org identity and core-repo allowlist via the shared library, then invokes report.py --repos scoped to exactly those repos — plus a hermetic test covering arg-building, date-range pass-through, and fail-loud on a missing generator.

All checks pass. Ready for human review.


Reviewed by clawgenti using the github-pr-review skill

@rubambiza rubambiza self-assigned this Aug 18, 2026
@rubambiza rubambiza added ready-for-human-review AI review passed, ready for human reviewer and removed ready-for-ai-review Request automated AI code review from clawgenti labels Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human-review AI review passed, ready for human reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Scope weekly report to core repos

2 participants