Add Solari Systems LLC to CLA signers #511
Conversation
📝 WalkthroughWalkthroughThis PR updates the CLA signers registry by adding a new corporation entry for "Solari Systems LLC" and refreshing the last_updated timestamp to 2026-01-04. The corporation entry includes contact information, signed date, CLA version, and associated GitHub usernames and email addresses. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
CLA Verification FailedThe following contributors have not signed the Contributor License Agreement:
How to Sign
This check runs automatically. Maintainers can update |
|
Thanks for the opportunity to resubmit. This follows the same pattern as #401, which worked end-to-end. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Fix all issues with AI Agents 🤖
In @.github/cla-signers.json:
- Around line 189-201: The "Solari Systems LLC" CLA entry violates the corporate
template: replace the contact_name value (currently the company name) with an
individual Point of Contact name, replace contact_email and emails from the
personal gmail address with an official corporate email using the company's
domain, populate the domains array with the company's verified domains (e.g.,
"solarisystems.com"), add the required Point of Contact Phone field
(contact_phone) per CLA-CORPORATE.md, and ensure signed_date/cla_version remain
correct only after confirming the entity is a legitimate registered company and
the provided contact details are authorized.
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/cla-signers.json
🔇 Additional comments (1)
.github/cla-signers.json (1)
3-3: LGTM! Timestamp properly updated.The last_updated timestamp correctly reflects the date of this update and matches the signed_date of the new corporation entry.
.github/cla-signers.json
Outdated
| "Solari Systems LLC": { | ||
| "contact_name": "Solari Systems LLC", | ||
| "contact_email": "solarisys2025@gmail.com", | ||
| "signed_date": "2026-01-04", | ||
| "cla_version": "1.0", | ||
| "domains": [], | ||
| "github_usernames": [ | ||
| "SolariSystems" | ||
| ], | ||
| "emails": [ | ||
| "solarisys2025@gmail.com" | ||
| ] | ||
| } |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
# First, let's read the cla-signers.json file to verify the entry
cat -n .github/cla-signers.json | sed -n '180,210p'Repository: cortexlinux/cortex
Length of output: 743
🏁 Script executed:
# Check the full structure of a few corporate entries for comparison
jq '.[] | select(.contact_name != .github_usernames[0]? and .domains != null) | {name: . | keys[0], contact_name, contact_email, domains: .domains}' .github/cla-signers.json | head -50Repository: cortexlinux/cortex
Length of output: 157
🏁 Script executed:
# Search for CLA policy documents and validation rules
find . -type f \( -name "*CLA*" -o -name "*cla*" -o -name "*CONTRIBUTOR*" \) | grep -v node_modules | head -20Repository: cortexlinux/cortex
Length of output: 297
🏁 Script executed:
# Check if there are any CLA-related validation scripts or documentation
rg -i "cla|contributor.*license" --type md --type txt --type json -l | grep -v node_modules | head -20Repository: cortexlinux/cortex
Length of output: 622
🏁 Script executed:
# Read the corporate CLA policy
cat ./CLA-CORPORATE.mdRepository: cortexlinux/cortex
Length of output: 2502
🏁 Script executed:
# Read the general CLA policy
cat ./CLA.mdRepository: cortexlinux/cortex
Length of output: 3110
🏁 Script executed:
# Read the CLA validation script
cat ./.github/scripts/cla_check.pyRepository: cortexlinux/cortex
Length of output: 13303
🏁 Script executed:
# Get the full structure to see patterns in corporate entries
jq 'to_entries[] | select(.value | type == "object" and has("domains")) | {name: .key, contact_name: .value.contact_name, contact_email: .value.contact_email, domains: .value.domains}' .github/cla-signers.jsonRepository: cortexlinux/cortex
Length of output: 44
This corporate entry violates the CLA-CORPORATE.md template requirements.
Solari Systems LLC fails to comply with the documented corporate CLA policy in multiple ways:
-
contact_name violation: The template requires "Point of Contact Name" (an individual), not the corporation name. Compare with "Mike Morgan" for AI Venture Holdings LLC.
-
Gmail address instead of corporate domain: Corporate CLAs require professional contact information. Using "solarisys2025@gmail.com" is inconsistent with corporate signer expectations.
-
Empty domains array: The corporate CLA should include verified domains (like "aiventureholdings.com" and "cortexlinux.com" for AI Venture Holdings). An empty array means no domain-based verification is possible and the entry relies solely on exact username/email matching.
-
Missing contact phone: CLA-CORPORATE.md explicitly requests "Point of Contact Phone" in Section 2.
-
Email metadata: The "2025" suffix in the Gmail address suggests a very recent account creation.
Before merging, verify that Solari Systems LLC is a legitimate registered entity with proper authorization, and update the entry to comply with the corporate CLA template (individual contact name, corporate email domain, and verified domains).
🤖 Prompt for AI Agents
In @.github/cla-signers.json around lines 189-201, The "Solari Systems LLC" CLA
entry violates the corporate template: replace the contact_name value (currently
the company name) with an individual Point of Contact name, replace
contact_email and emails from the personal gmail address with an official
corporate email using the company's domain, populate the domains array with the
company's verified domains (e.g., "solarisystems.com"), add the required Point
of Contact Phone field (contact_phone) per CLA-CORPORATE.md, and ensure
signed_date/cla_version remain correct only after confirming the entity is a
legitimate registered company and the provided contact details are authorized.
This reverts commit 4396b65.
|
1 participant
Adds Solari Systems LLC to the corporate CLA signers list as requested in #496. Follows the pattern used in #401.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.