feat: Introduce core TUI spreadsheet application with AI features, da…#1
feat: Introduce core TUI spreadsheet application with AI features, da…#1SreeAditya-Dev wants to merge 1 commit intomainfrom
Conversation
…ta cleaning, charting, and comprehensive project templates. Updates repository and organization names to SreeAditya-Dev/Cello-TUI - Renames the project from CodeOne45/cello-tui to SreeAditya-Dev/Cello-TUI across all configuration files, documentation, CI workflows, and import paths. - Adjusts Homebrew tap metadata, changelog entries, and installer URLs to reference the new organization name. - Updates module declaration in go.mod and internal package imports to use the new module path. - This refactor aligns the codebase with the new maintainer/organization branding and ensures consistent imports and reference links throughout the repository.
There was a problem hiding this comment.
Pull request overview
This PR renames the repository from CodeOne45/cello-tui to SreeAditya-Dev/Cello-TUI, updates all references throughout the codebase, and adds several open-source project templates and documentation files.
Changes:
- Updates Go module path and all internal imports from CodeOne45/cello-tui to SreeAditya-Dev/Cello-TUI
- Updates documentation URLs (INSTALL.md, CONTRIBUTING.md, SECURITY.md) to reference the new organization
- Adds CODE_OF_CONDUCT.md, OSS_SETUP_GUIDE.md, and GitHub issue/PR templates
- Updates CHANGELOG.md to reflect version 1.0.0 as initial release under new organization
- Updates .goreleaser.yml Homebrew tap configuration for new owner
- Changes .gitignore from "vex" to "cello" binary name
- Completely replaces sample data from product sales to employee data
Reviewed changes
Copilot reviewed 25 out of 26 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| go.mod | Updates module declaration to github.com/SreeAditya-Dev/Cello-TUI with Go 1.25 |
| main.go | Updates import paths and help text URLs to new repository |
| internal/ui/ui.go | Updates import paths to new module path |
| internal/loader/save.go | Updates import paths to new module path |
| internal/loader/loader.go | Updates import paths to new module path |
| internal/chart/chart.go | Updates import paths to new module path |
| internal/app/view.go | Updates import paths to new module path |
| internal/app/update.go | Updates import paths to new module path |
| internal/app/model.go | Updates import paths to new module path |
| internal/app/formulas.go | Updates import paths to new module path |
| internal/app/edit.go | Updates import paths to new module path |
| internal/app/config_tui.go | Updates import paths to new module path |
| internal/ai/formula_generator.go | Updates import paths to new module path |
| examples/sample_data.csv | Completely replaces sample data from products to employees |
| SECURITY.md | Updates GitHub Security Advisory URL and adds version 2.1.x support |
| OSS_SETUP_GUIDE.md | Adds new guide for open source repository setup |
| INSTALL.md | Updates all installation URLs and commands to new repository |
| CONTRIBUTING.md | Updates clone and upstream URLs to new repository |
| CODE_OF_CONDUCT.md | Adds Contributor Covenant Code of Conduct (with placeholder email) |
| CHANGELOG.md | Simplifies to show only version 1.0.0 with new repository URL |
| .goreleaser.yml | Updates Homebrew tap owner to SreeAditya-Dev |
| .gitignore | Changes binary name from "vex" to "cello" |
| .github/pull_request_template.md | Adds new PR template with hardware-focused test config |
| .github/ISSUE_TEMPLATE/feature_request.yml | Adds feature request template with placeholder assignee |
| .github/ISSUE_TEMPLATE/config.yml | Adds issue template configuration |
| .github/ISSUE_TEMPLATE/bug_report.yml | Adds bug report template with browser fields and placeholder assignee |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| assignees: | ||
| - octocat |
There was a problem hiding this comment.
The placeholder assignee "octocat" should be replaced with an actual GitHub username of a maintainer or removed if no default assignee is needed. This is a generic GitHub placeholder that won't route issues to the correct person.
| assignees: | |
| - octocat |
| assignees: | ||
| - octocat |
There was a problem hiding this comment.
The placeholder assignee "octocat" should be replaced with an actual GitHub username of a maintainer or removed if no default assignee is needed. This is a generic GitHub placeholder that won't route issues to the correct person.
| assignees: | |
| - octocat |
| @@ -1,4 +1,4 @@ | |||
| module github.com/CodeOne45/cello-tui | |||
| module github.com/SreeAditya-Dev/Cello-TUI | |||
There was a problem hiding this comment.
The module path uses "Cello-TUI" with capital letters, but this creates an inconsistency with the repository name casing. GitHub repositories are typically lowercase with hyphens (e.g., "cello-tui"), and while GitHub URLs are case-insensitive, Go module paths are case-sensitive. This means users following documentation that references "cello-tui" (lowercase) for go install commands may encounter issues. Consider using lowercase "cello-tui" in the module path to match standard Go conventions and ensure consistency across all documentation and tooling.
| module github.com/SreeAditya-Dev/Cello-TUI | |
| module github.com/SreeAditya-Dev/cello-tui |
| id: browsers | ||
| attributes: | ||
| label: What browsers are you seeing the problem on? | ||
| multiple: true | ||
| options: | ||
| - Firefox | ||
| - Chrome | ||
| - Safari | ||
| - Microsoft Edge |
There was a problem hiding this comment.
The bug report template includes a "What browsers are you seeing the problem on?" field with browser options (Firefox, Chrome, Safari, Microsoft Edge). This is not applicable for a terminal-based TUI application. Consider replacing this with a more relevant field such as "Terminal emulator" with options like iTerm2, Alacritty, Windows Terminal, GNOME Terminal, etc.
| id: browsers | |
| attributes: | |
| label: What browsers are you seeing the problem on? | |
| multiple: true | |
| options: | |
| - Firefox | |
| - Chrome | |
| - Safari | |
| - Microsoft Edge | |
| id: terminal-emulator | |
| attributes: | |
| label: Which terminal emulator are you using? | |
| multiple: true | |
| options: | |
| - iTerm2 | |
| - Alacritty | |
| - Windows Terminal | |
| - GNOME Terminal | |
| - Kitty | |
| - Other |
| - Firmware version: | ||
| - Hardware: | ||
| - Toolchain: | ||
| - SDK: |
There was a problem hiding this comment.
The test configuration section includes fields for "Firmware version", "Hardware", "Toolchain", and "SDK" which appear to be copied from an embedded systems or hardware project template. For a Go-based TUI application, more relevant fields would be: "Go version", "Operating System", "Terminal emulator", and "Shell".
| - Firmware version: | |
| - Hardware: | |
| - Toolchain: | |
| - SDK: | |
| - Go version: | |
| - Operating System: | |
| - Terminal emulator: | |
| - Shell: |
| ID,Name,Department,Role,Salary,Join Date,Active | ||
| EMP001,John Smith,Engineering,Senior Developer,125000,2023-01-15,TRUE | ||
| EMP002,Sarah Johnson,Marketing,Head of Growth,115000,2023-02-01,TRUE | ||
| EMP003,Michael Brown,Engineering,DevOps Engineer,130000,2023-01-20,TRUE | ||
| EMP004,Emily Davis,Design,Lead Designer,105000,2023-03-10,TRUE | ||
| EMP005,David Wilson,Product,Product Manager,120000,2023-02-15,FALSE | ||
| EMP006,Jessica Miller,Sales,Account Executive,95000,2023-04-01,TRUE | ||
| EMP007,James Taylor,Engineering,Frontend Dev,110000,2023-03-05,TRUE | ||
| EMP008,Lisa Anderson,Marketing,Content Strategist,85000,2023-05-12,TRUE | ||
| EMP009,Robert Martin,Finance,Financial Analyst,90000,2023-04-20,TRUE | ||
| EMP010,Jennifer White,HR,HR Manager,100000,2023-01-10,TRUE |
There was a problem hiding this comment.
The sample data has been completely replaced from product sales data (Laptop Pro, Wireless Mouse, etc.) to employee data (John Smith, Sarah Johnson, etc.). This significant change to the example dataset is not mentioned in the PR description, which focuses solely on the repository rename. If this change is intentional, please explain the rationale in the PR description. If it's unintentional, this change should be reverted as it's unrelated to the repository rename.
| ## Enforcement | ||
|
|
||
| Instances of abusive, harassing, or otherwise unacceptable behavior may be | ||
| reported to the community leaders responsible for enforcement at [INSERT EMAIL ADDRESS]. |
There was a problem hiding this comment.
The placeholder "[INSERT EMAIL ADDRESS]" needs to be replaced with an actual contact email address for reporting Code of Conduct violations. This is a required field that community members will need to report enforcement issues.
| reported to the community leaders responsible for enforcement at [INSERT EMAIL ADDRESS]. | |
| reported to the community leaders responsible for enforcement at coc@example.com. |
| ```bash | ||
| # Download latest release | ||
| curl -L https://github.com/CodeOne45/cello-tui/releases/latest/download/cello-darwin-arm64.tar.gz -o cello-tui.tar.gz | ||
| curl -L https://github.com/SreeAditya-Dev/Cello-TUI/releases/latest/download/cello-darwin-arm64.tar.gz -o cello-tui.tar.gz |
There was a problem hiding this comment.
The install instructions use curl to download and extract a prebuilt binary archive directly from GitHub Releases without any integrity verification (checksum or signature). If the release artifact or distribution channel were compromised, users following these steps could execute malicious code as their local user. Add an integrity check step (for example, publishing and verifying a cryptographic checksum or signature for the archive) before extraction/execution to harden against supply chain attacks.
| ```bash | ||
| # Download latest release | ||
| wget https://github.com/CodeOne45/cello-tui/releases/latest/download/cello-linux-amd64.tar.gz | ||
| wget https://github.com/SreeAditya-Dev/Cello-TUI/releases/latest/download/cello-linux-amd64.tar.gz |
There was a problem hiding this comment.
The Linux install instructions use wget to download a prebuilt binary archive from GitHub Releases with no accompanying checksum or signature verification before extraction and use. If an attacker managed to tamper with the release artifact or delivery path, users following these commands could run arbitrary malicious code. Consider publishing and requiring verification of a cryptographic checksum or signature for the downloaded archive before extraction to mitigate this supply chain risk.
…ta cleaning, charting, and comprehensive project templates.
Updates repository and organization names to SreeAditya-Dev/Cello-TUI