feat : clean the whole codebase

[bella-247]

This pull request refactors and clarifies the codebase, particularly in the PostgreSQL parser, and significantly improves the contributing documentation. The main changes include enhanced variable naming for clarity, modularization of the schema linking logic, and a major rewrite of the contribution guide to focus on maintainability, explicit standards, and clearer instructions for contributors.

Documentation and Contribution Guide Improvements:

• The CONTRIBUTING.md file was rewritten to emphasize core design principles (explicit naming, SRP, code style), clarify architectural/testing rules, and provide a more maintainable and readable structure. It replaces the "types of contributions" section with design principles, details code and architectural rules, and updates the workflow to use go build/go test instead of make. Conventional Commit guidelines are now included. [1] [2] [3] [4] [5] [6]

Parser Code Refactoring and Clarity:

• All variable and parameter names in internal/parser/postgresql/adapter_cgo.go were rewritten to be more explicit and descriptive, replacing short/cryptic names with clear, full-word alternatives (e.g., stmt → statement, col → column, tc → tableConstraint). This improves code readability and maintainability. [1] [2] [3] [4]
• Helper functions were clarified and renamed for intent, e.g., fkActionString → mapForeignKeyAction, and all switch/case variables use descriptive names.

Schema Linking Logic Modularization:

• The schema linking logic in internal/parser/postgresql/link.go was refactored: the monolithic link method was split into smaller, focused helpers (buildTableIndex, resolveForeignKeys) for better separation of concerns and readability.

These changes collectively improve code quality, contributor experience, and future maintainability.## What does this PR do?

Why is it needed?

Closes #

How was it tested?

Does it change any existing behavior?

Checklist

• Tests added or updated
• make test passes (go test ./... -race)
• make lint passes (golangci-lint run ./...)
• No cross-stage imports (parser doesn't import generator, etc.)
• No time.Now() or unseeded randomness in the generation pipeline
• Error messages follow the format in SRS §12.2
• Exported functions have doc comments
• Golden files updated if output format changed (make test-golden UPDATE=true)

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: a60291c9-3976-47d7-9be1-5da7e3005b74

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

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