fix: anonymize all personal data from framework codebase

[simongonzalezdc]

Removes all personal identifiers that leaked into the public framework:

Code defaults:

• cli.py: --owner default was Pastorsimon1798 → now reads from ARCHAEOLOGY_GITHUB_OWNER env var
• github_fetcher.py: same _DEFAULT_OWNER fix
• agent_benchmark.py: normalize_author() returned 'Simon' for any human author → now returns 'Human' for all non-AI names; CSS/JS token --simon → --human

References in source:

• api.py: removed 'The-Factory' from module docstring
• db/queries.py: removed 'Liminal case study' from comment
• report.py: removed 'Liminal' from demo README text

Demo deliverables:

• Both deliverables/archaeology.html and deliverables/visuals/archaeology.html had a callout with 'Jake Van Clief', 'Simon's first-ever PR', and 'mcp-video' — replaced with anonymized generic equivalent

---

^{Need help on this PR? Tag @codesmith with what you need. Autofix is disabled.}

Summary by CodeRabbit

• Configuration

  • GitHub owner configuration now uses environment variable (ARCHAEOLOGY_GITHUB_OWNER) for improved flexibility instead of hardcoded defaults.

• Visualization

  • Agent benchmark visualizations simplified with generic "Human" agent labeling and updated color theming.

• Content Updates

  • Removed product-specific references from generated reports and case studies.
  • Updated privacy exclusion descriptions in exported documentation.
  • Demo project content updated with context methodology insights instead of person-specific credits.
  • Improved database schema compatibility documentation.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1bc7cd87-eee8-4347-93ec-be9707627872

📥 Commits

Reviewing files that changed from the base of the PR and between f8e74ce and 56ca71d.

📒 Files selected for processing (8)

• archaeology/api.py
• archaeology/cli.py
• archaeology/db/queries.py
• archaeology/report.py
• archaeology/visualization/agent_benchmark.py
• archaeology/visualization/github_fetcher.py
• projects/demo-project/deliverables/archaeology.html
• projects/demo-project/deliverables/visuals/archaeology.html

---

📝 Walkthrough

Walkthrough

This PR removes hardcoded references to a specific person (Simon) and product (Liminal/The-Factory) across multiple modules. Configuration defaults shift from hardcoded to environment variables, agent visualization logic treats unknown authors as generic "Human" instead of preserving names, and documentation is updated to use neutral language.

Changes

Depersonalization and Generalization

Layer / File(s)	Summary
GitHub owner configuration externalization archaeology/visualization/github_fetcher.py, archaeology/cli.py	GitHub owner defaults moved from hardcoded "Pastorsimon1798" to environment variable ARCHAEOLOGY_GITHUB_OWNER with empty-string fallback; fetch-github CLI option, dashboard command, and publish-static command owner_labels mappings updated accordingly.
Agent benchmark author labeling and visualization archaeology/visualization/agent_benchmark.py	Author normalization treats non-AI authors as generic "Human" label; CSS theme token changed from --simon to --human; JavaScript color extraction and agent-to-color mapping updated; metrics table rework-rate color logic uses colors.human instead of colors.simon.
API and database documentation updates archaeology/api.py, archaeology/db/queries.py	Module docstring changed to refer to "external consumers" instead of "The-Factory"; get_eras docstring updated to document fallback behavior across schema versions.
Case study and project documentation narratives archaeology/report.py, projects/demo-project/deliverables/archaeology.html, projects/demo-project/deliverables/visuals/archaeology.html	Case study README content and demo project callouts changed from "KEY PERSON — Jake Van Clief" to "KEY INSIGHT — Context Methodology"; "Liminal" and "YouTube export" references removed in favor of generalized descriptions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/anonymize-personal-data

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 56ca71d3f4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Require an owner before calling gh

When ARCHAEOLOGY_GITHUB_OWNER is unset and the user runs archaeology fetch-github without --owner, this default becomes "", but the fetcher still passes it as a positional owner to gh repo list rather than omitting the optional argument. The GitHub CLI manual documents the syntax as gh repo list [<owner>], so defaulting to an explicit empty string can make the default command fail or fetch no repos instead of either using the authenticated user/default behavior or giving an actionable error; validate this value or omit the positional argument when it is blank.

Useful? React with 👍 / 👎.