Skip to content

GHCP -- Run: Ex1 Repo Orientation and Architecture Map - #3592

Closed
neha-p6 wants to merge 1 commit into
mainfrom
learn/run/neha-p6-ex1
Closed

GHCP -- Run: Ex1 Repo Orientation and Architecture Map#3592
neha-p6 wants to merge 1 commit into
mainfrom
learn/run/neha-p6-ex1

Conversation

@neha-p6

@neha-p6 neha-p6 commented May 22, 2026

Copy link
Copy Markdown

Summary

  • Added scripts/refresh-diagram.sh: scans components/ for Go modules and commands/ source files, diffs against ai-track-docs/architecture.mmd nodes, and emits a labelled drift report with counts.
  • Added .github/workflows/refresh-diagram.yml: triggers on PRs touching components/**/*.go; runs the refresh script; posts the drift report to the job summary (advisory, never blocks).
  • Updated ai-track-docs/architecture.mmd: added http_resilience.go node and edge (Walk Ex15 addition previously missing from diagram).

Patch plan: new advisory drift-check script + CI workflow + one diagram node addition. Zero behavior change to production code.

Files touched:

  • scripts/refresh-diagram.sh (new)
  • .github/workflows/refresh-diagram.yml (new)
  • ai-track-docs/architecture.mmd (updated)

Evidence

Before (old diagram, pre-this-PR):

http_resilience.go   ✗ MISSING
Summary: 1 item(s) found in codebase but not in diagram

After (bash scripts/refresh-diagram.sh on updated diagram):

### Component modules vs diagram nodes
  components/chef-automate-collect   ✓ matched
  components/main-chef-wrapper       ✓ matched

### commands/ source files vs diagram nodes
  automate_config.go       ✓ matched
  describe.go              ✓ matched
  environment_variables.go ✓ matched
  http_resilience.go       ✓ matched   ← newly added
  init_cli.go              ✓ matched
  cli_io.go                ✗ MISSING   (known backlog – diagram covers key files)
  gen_config.go            ✗ MISSING
  metadata_collectors.go   ✗ MISSING
  report_new_rollout.go    ✗ MISSING
  show_config.go           ✗ MISSING
  test_config.go           ✗ MISSING

### Summary
  Component modules : 2 matched, 0 missing
  commands/ files   : 5 matched, 6 missing (drift surfaced for future PRs)
  ✓  http_resilience.go drift cleared by this PR

Delegation checklist completed: yes

Risk & Rollback

  • Risk: low
  • Rollback: git revert 62846531c014f8f4c4309ba22488617332d6bbf4
  • Rollback commands:
    git revert 62846531c014f8f4c4309ba22488617332d6bbf4
    git push origin learn/run/neha-p6-ex1
    
  • No production code changed; script and workflow are fully additive and removable.

Track

  • Level: Run
  • Exercise: Ex1

Signed-off-by: neha-p6 <neha.pansare@progress.com>
@neha-p6
neha-p6 force-pushed the learn/run/neha-p6-ex1 branch from 90f2680 to 6284653 Compare May 22, 2026 07:19
@neha-p6 neha-p6 closed this May 22, 2026
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.

1 participant