Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/dna assistant #3192

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

theRealThomasDavidson
Copy link

What kind of change does this PR introduce?

Bug fix, feature, docs update, ...

What is the current behavior?

Please link any relevant issues here.

What is the new behavior?

Feel free to include screenshots if it includes visual changes.

Additional context

Add any other context or screenshots.

Core Features:
- Interactive chat assistant with OpenAI integration
- Schema analysis and normalization tools
- Supabase documentation search
- CLI doctor and configuration

UI Improvements:
- Semantic color coding
  - Banner: Green
  - Assistant: Cyan
  - Commands: Yellow
  - Tool output: Coral (F88379)
  - Errors: Red
  - Separators: Light Black

Infrastructure:
- Add DNA internal package
- Update dependencies
- Add testing scripts
- Integrate with CLI bootstrap
- Add PowerShell and Bash build scripts for cross-platform builds
- Fix line endings in migration files to use native OS format
- Add runtime detection for proper line endings in SQL files
- Update migration tools to handle line endings consistently

Build scripts support both Windows (build.ps1) and Unix (build.sh) systems.
Line ending fixes prevent PostgreSQL migration errors.
@theRealThomasDavidson theRealThomasDavidson requested a review from a team as a code owner February 23, 2025 23:10
migrationConfig.Port,
migrationConfig.Database)

fmt.Printf("Attempting to connect to: %s\n", strings.Replace(connStr, migrationConfig.Password, "****", 1))

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information High

Sensitive data returned by an access to Password
flows to a logging call.
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