Skip to content

Conversation

@ericelliott
Copy link
Collaborator

@ericelliott ericelliott commented Dec 14, 2025

Update all references to "SudoLang AIDD" and "SudoLang.ai" to use
"AIDD Framework" as the project title, while preserving references
to SudoLang as the prompt language.


Note

Rebrands project to AIDD Framework, updates CLI help text and tests, fixes docs links, and ignores core dumps.

  • Branding & Docs
    • Rename "SudoLang AIDD"/"SudoLang.ai" to "AIDD Framework" across README.md, activity-log.md, and plan.md.
    • Update badges, headings, and TOC entries in README.md; preserve SudoLang language description.
    • Fix link in docs/server/README.md to point to https://github.com/paralleldrive/aidd.
  • CLI
    • Update bin/aidd.js description and help text to AIDD Framework branding.
    • Replace "About the Author" section with ParallelDrive CTA; refine Quick Start and AI workflow command guidance.
  • Tests
    • Adjust bin/cli-help-e2e.test.js to assert new branding, CTA placement, Quick Start structure, and workflow command list.
  • Repo hygiene
    • Add core dump patterns (core, core.*) to .gitignore.

Written by Cursor Bugbot for commit 29066a9. This will update automatically on new commits. Configure here.

Update all references to "SudoLang AIDD" and "SudoLang.ai" to use
"AIDD Framework" as the project title, while preserving references
to SudoLang as the prompt language.
Copilot AI review requested due to automatic review settings December 14, 2025 16:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the project branding from "SudoLang AIDD" to "AIDD Framework" throughout the codebase, clarifying that SudoLang refers specifically to the prompt language used within the framework, not the framework itself.

Key Changes:

  • Updated project title from "SudoLang AIDD" to "AIDD Framework" across all documentation and code
  • Preserved references to "SudoLang" as the prompt language component
  • Updated CLI help text, test assertions, and documentation to reflect the new naming

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
plan.md Updated project plan title to "AIDD Framework Project Plan"
bin/cli-help-e2e.test.js Updated test assertions to check for "AIDD Framework" instead of "SudoLang AIDD"
bin/aidd.js Updated CLI description, help text, and author attribution to use "AIDD Framework"
activity-log.md Updated July 6, 2025 activity log entry to reference "AIDD Framework project overview"
README.md Updated main title, badge, heading, and description to use "AIDD Framework" while preserving SudoLang references as the prompt language

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Simplify CLI description by removing redundant text
- Replace EricElliottJS CTA with ParallelDrive.com
- Update tests to reflect new CTA content
@ericelliott
Copy link
Collaborator Author

@cursoragent please /review

@cursor
Copy link

cursor bot commented Dec 14, 2025

🔬 Code Review: Rename SudoLang AIDD to AIDD Framework

🎯 Summary

This PR successfully rebrands "SudoLang AIDD" to "AIDD Framework" across documentation and CLI, while correctly preserving references to SudoLang as the prompt language. The changes include two commits with thoughtful updates to help text and call-to-action sections.


✅ Strengths

1. Thorough and Consistent Rebranding

  • Successfully renamed project in all key locations: README.md, plan.md, activity-log.md, CLI description, and test assertions
  • Correctly preserved "SudoLang" references as the prompt language (e.g., "SudoLang is a pseudocode language...")
  • Updated badge text and TOC anchors appropriately

2. Improved CLI Help Text

  • Simplified description from "Install AIDD Framework AI agent orchestration system" to cleaner "Install the AIDD Framework"
  • Replaced verbose "About the Author" section with concise ParallelDrive CTA
  • Better alignment with professional open-source project presentation

3. Comprehensive Test Updates

  • All E2E test assertions properly updated to match new text
  • Test descriptions clearly reflect the changes
  • Maintains test coverage across help text sections

4. Clean Commit Structure

  • Two logical commits: (1) core rebrand, (2) marketing/CTA updates
  • Clear, descriptive commit messages following conventional format
  • Appropriate scope of changes per commit

⚠️ Issues Requiring Attention

1. CRITICAL: Incomplete Rebrand - Stale Reference Found

See the main [AIDD repository](https://github.com/paralleldrive/sudolang.ai) for details.

Impact: External link points to outdated sudolang.ai repository URL instead of aidd
Fix Required: Update to https://github.com/paralleldrive/aidd

2. MEDIUM: Archived Documentation Contains Stale References

The file /tasks/archive/2025-09-27-aidd-cli-epic.md contains two references to "sudolang.ai" package:

  • Line 12: "existing sudolang.ai package"
  • Line 29: "existing sudolang.ai package"

Recommendation: While this is an archived epic, consider updating for historical accuracy or add a note that the package was later renamed to aidd. This prevents confusion if someone reviews project history.


📋 Detailed Analysis

Code Structure & Organization ✅

  • Changes are minimal, focused, and non-breaking
  • No structural modifications to logic
  • Clear separation between documentation and code changes

Coding Standards & Best Practices ✅

  • No code logic changes - only string literals updated
  • Test assertions follow existing patterns
  • Maintains consistency with project conventions

Test Coverage & Quality ✅

  • All affected tests updated appropriately
  • Test descriptions accurately reflect new expectations
  • No test gaps introduced
  • E2E tests validate the complete help output flow

Performance Considerations ✅

  • Zero performance impact (string literal changes only)
  • No algorithmic or runtime changes

Security Analysis ✅

OWASP Top 10 Review: None applicable to this PR (documentation/branding only)

  • ✅ No injection vulnerabilities
  • ✅ No authentication/authorization changes
  • ✅ No sensitive data exposure
  • ✅ No XML external entities
  • ✅ No broken access control
  • ✅ No security misconfigurations
  • ✅ No XSS vulnerabilities
  • ✅ No insecure deserialization
  • ✅ No components with known vulnerabilities
  • ✅ No insufficient logging/monitoring changes

Architectural Patterns ✅

  • No architectural changes
  • Maintains existing CLI structure
  • Follows Commander.js patterns appropriately

Documentation Quality ⚠️

  • README.md: Excellent updates, clear and consistent
  • CLI help text: Improved readability and professionalism
  • Commit messages: Clear and follow conventional commits format
  • Missing: The docs/server/README.md link needs correction (see Issue feat: Establish AIDD metaprograms system for Cursor #1)

🔍 Additional Observations

1. Positive Changes in Second Commit
The replacement of "About the Author" with a concise ParallelDrive CTA is a good decision:

  • More professional for an open-source framework
  • Reduces help output verbosity
  • Clear call-to-action without being overly promotional

2. Well-Structured Help Text
The CLI help now includes:

  • Framework description ✅
  • SudoLang explanation ✅
  • Server framework callout ✅
  • AI workflow commands ✅
  • Quick start instructions ✅
  • ParallelDrive CTA ✅

3. No Dead Code or Redundancies

  • Clean removal of old text
  • No leftover references
  • No unused test cases

🎯 Action Items

MUST FIX (Blocking):

  1. Update GitHub repository link in docs/server/README.md:1196
    # Current (incorrect):
    See the main [AIDD repository](https://github.com/paralleldrive/sudolang.ai) for details.
    
    # Should be:
    See the main [AIDD repository](https://github.com/paralleldrive/aidd) for details.

SHOULD FIX (Recommended):
2. 📝 Consider updating archived epic references

  • File: tasks/archive/2025-09-27-aidd-cli-epic.md
  • Add note: "Note: Package was later renamed from sudolang.ai to aidd"
  • Or update references to use current package name

NICE TO HAVE (Optional):
3. 🔍 Verify no additional external links need updating

  • Check if any external documentation or websites reference the old name
  • Update any social media or marketing materials

✨ Conclusion

Overall Assessment: Strong rebrand execution with one critical fix needed

Approval Status: ⚠️ Changes requested - Fix the docs/server/README.md link before merging

Quality Score: 9/10

  • Comprehensive coverage of rebrand
  • Clean, focused commits
  • Excellent test updates
  • One overlooked reference prevents full score

Once the docs/server/README.md link is corrected, this PR will be ready to merge. The rebrand is otherwise complete, professional, and well-executed.


Open in Cursor Open in Web

Learn more about Cursor Agents

Copilot AI review requested due to automatic review settings December 14, 2025 17:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@cursor
Copy link

cursor bot commented Dec 14, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@ericelliott
Copy link
Collaborator Author

@cursoragent please review again

@cursor
Copy link

cursor bot commented Dec 14, 2025

🔬 Code Review: Second Review - Critical Issue Found

🎯 Updated Status

Previous Issue: ✅ RESOLVED - The docs/server/README.md link has been correctly updated to https://github.com/paralleldrive/aidd

New Critical Issue: 🚨 BLOCKER FOUND - Accidental core dump committed to repository


🚨 CRITICAL: Core Dump File Committed

Issue: A 35MB core dump file was accidentally committed in commit c86ea7f

commit c86ea7f6864d4979e5ab90646808fea5ae28a248
Author: Cursor Agent <cursoragent@cursor.com>
Date:   Sun Dec 14 17:04:08 2025 +0000

    feat: Add core functionality
    
    Co-authored-by: support <support@paralleldrive.com>

 core | Bin 0 -> 36495360 bytes

File Details:

  • Path: /workspace/core
  • Size: 35MB (36,495,360 bytes)
  • Type: ELF 64-bit LSB core file from 'light-locker'
  • Impact:
    • Bloats repository permanently (even after removal, it stays in git history)
    • Security risk: core dumps may contain sensitive data from memory
    • Will be included in npm package if published
    • Violates best practices for repository hygiene

🔧 Required Fix

You must remove this file before merging. Here's how:

Option 1: Remove from this branch (Recommended)

# Remove the file and the commit that added it
git rm core
git commit --amend -m "Fix stale repository link in server docs"

# Force push to update the PR
git push origin HEAD --force

Option 2: Interactive rebase (if you want to preserve commit history)

# Remove the file
git rm core

# Commit the removal
git commit -m "Remove accidentally committed core dump"

# Rebase to drop the "Add core functionality" commit
git rebase -i HEAD~3  # Then mark c86ea7f as 'drop'

Then update .gitignore to prevent this in the future:

# Add to .gitignore
echo "core" >> .gitignore
echo "core.*" >> .gitignore
git add .gitignore
git commit -m "chore: Add core dump files to .gitignore"

✅ What's Good in This Update

1. Documentation Fix Applied Correctly

See the main [AIDD repository](https://github.com/paralleldrive/aidd) for details.

✅ Previous critical issue resolved

2. Minimal, Focused Change

  • Single line update in commit dea518c
  • Clean commit message
  • No unintended side effects (except for the core file issue)

📋 Remaining Minor Issues

Archived Epic References (Low Priority)
The file /tasks/archive/2025-09-27-aidd-cli-epic.md still contains references to sudolang.ai package on lines 12 and 29. While not blocking, consider adding a note for historical clarity.


🎯 Final Verdict

Approval Status: 🚨 CHANGES REQUIRED - BLOCKING ISSUE

Critical Action Required:

  1. MUST: Remove the core dump file and clean git history
  2. MUST: Add core and core.* to .gitignore
  3. DONE: Repository link fix is correct

Once the core dump is removed, the rebrand PR will be clean and ready to merge. The documentation updates are excellent and the renaming has been executed thoroughly.


Summary

The repository link fix looks perfect, but the accidental inclusion of a 35MB core dump file is a blocking issue that must be addressed before merging. This appears to have been an unintended consequence of the fix process. Please remove the core file and update the PR.


Open in Cursor Open in Web

Learn more about Cursor Agents

Copilot AI review requested due to automatic review settings December 14, 2025 17:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 6 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Also add core dump files to .gitignore to prevent accidental commits.
@ericelliott ericelliott force-pushed the claude/rename-sudolang-aidd-V4wX2 branch from 568f29f to 29066a9 Compare December 14, 2025 17:25
@ericelliott
Copy link
Collaborator Author

@cursoragent please review again

@cursor
Copy link

cursor bot commented Dec 14, 2025

🔬 Code Review: Third Review - Core Dump Partially Resolved

🎯 Current Status

Previous Critical Issue: ⚠️ PARTIALLY RESOLVED - Core dump removed from working tree but still in git history

Current State: The PR has 5 commits total, including 2 problematic ones related to the core dump


📊 Commit Analysis

The PR currently contains these commits:

  1. ed9f788 - Rename SudoLang AIDD to AIDD Framework (GOOD)
  2. 9cd4dce - Update CLI description and replace CTA with ParallelDrive (GOOD)
  3. 29066a9 - Fix stale repository link in server docs (GOOD)
  4. ⚠️ 83c16d8 - feat: Add core functionality (35MB core dump added - BAD)
  5. ⚠️ c5415b9 - Remove accidentally committed core dump (removal - CLEANUP)

⚠️ Remaining Issue: Git History Bloat

The Problem:
While the core dump file is no longer tracked and .gitignore has been updated, the 35MB binary blob still exists in the git history because commits #4 and #5 remain in the branch.

Evidence:

git rev-list --objects --all | grep "core$"
# Returns: 6b150672bba007ee4835b23fde0ad8b7dfffcf35 core

Impact:

  • When merged, this 35MB blob becomes permanent repository history
  • Every clone of the repo will download this unnecessary data
  • Cannot be removed without rewriting history (force push)

💡 Recommended Solutions

Option A: Squash on Merge (Easiest)
When merging this PR, use GitHub's "Squash and merge" option. This will:

Option B: Interactive Rebase (Clean but requires force push)

git rebase -i HEAD~5
# In the editor, mark commits 83c16d8 and c5415b9 as 'drop'
git push --force-with-lease

Option C: Accept as-is
If this is acceptable to the team, the current state is functional:

  • ✅ Working tree is clean
  • .gitignore prevents future incidents
  • ⚠️ History contains the blob (but only in this branch until merge)

✅ What's Correct Now

1. Branding Updates - Comprehensive and Complete ✅

# AIDD Framework

[![AIDD Framework](https://img.shields.io/badge/✨_AIDD_Framework-black)](https://github.com/paralleldrive/aidd)[![Parallel Drive](https://img.shields.io/badge/🖤_Parallel_Drive-000000?style=flat)](https://paralleldrive.com)

2. Repository Link Fixed

See the main [AIDD repository](https://github.com/paralleldrive/aidd) for details.

3. CLI Updates

  • Description simplified to "Install the AIDD Framework"
  • Replaced author bio with concise ParallelDrive CTA
  • All test assertions updated correctly

4. Repository Hygiene

# Core dump files
core
core.*

5. Test Coverage

  • All E2E tests pass with new branding
  • Assertions correctly updated for CTA and Quick Start placement

📋 Final Code Quality Assessment

Structure & Organization ✅

  • Clean, focused changes to branding
  • Logical commit progression (except core dump incident)
  • Proper separation of concerns

Documentation Quality ✅

  • README thoroughly updated
  • All references consistently changed
  • CLI help text improved

Testing ✅

  • Comprehensive E2E coverage
  • All assertions updated correctly
  • No test gaps

Security ✅

  • No vulnerabilities introduced
  • No sensitive data in changes (excluding the core dump blob)
  • OWASP Top 10: Not applicable (documentation changes only)

Repository Hygiene ⚠️

  • .gitignore properly updated
  • Working tree is clean
  • History still contains 35MB blob (see recommendations above)

🎯 Final Recommendation

Approval Status: ✅ APPROVE WITH SQUASH MERGE

Merge Strategy: Use "Squash and merge" when merging this PR to GitHub

Rationale:

  • The actual code changes are excellent and complete
  • All branding has been updated correctly
  • The core dump issue is isolated to the branch history
  • Squashing will eliminate the problematic commits from the permanent record
  • No force-push required, simpler workflow

📝 Minor Note: Archived Documentation

The file /tasks/archive/2025-09-27-aidd-cli-epic.md still contains references to sudolang.ai on lines 12 and 29. This is acceptable as it's archived historical documentation that reflects what the package was called when the epic was created.


✨ Summary

This is a well-executed rebrand with one hiccup during the fix process. The solution is simple: use squash merge when merging to main, which will create a clean single commit without the core dump in the permanent history.

Bottom line: The PR achieves its goals completely. The core dump incident has been mitigated, and proper squash merging will prevent it from affecting the main branch. Ready to merge with squash strategy. 🚀


Open in Cursor Open in Web

Learn more about Cursor Agents

@ericelliott ericelliott force-pushed the claude/rename-sudolang-aidd-V4wX2 branch from c5415b9 to 29066a9 Compare December 14, 2025 18:25
@ericelliott ericelliott merged commit 3906d49 into main Dec 14, 2025
13 checks passed
@ericelliott ericelliott deleted the claude/rename-sudolang-aidd-V4wX2 branch December 14, 2025 18:28
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.

3 participants