add socratic ai icons

[RealTejasRao]

Description

Added SVG icons for Socratic AI

Type

• New icon(s)
• Icon update/fix
• Bug fix
• New feature
• Documentation
• Maintenance/chore

Checklist

• My changes follow the project's conventions
• I've tested the build: cd packages/icons && npm run build
• Security audit passes: npm run audit
• Dist validation passes: npm run validate

For icon submissions:

• SVG has a valid viewBox attribute
• SVG is under 50KB
• No embedded <script> tags or event handlers
• Icon accurately represents the official brand
• icons.json entry is complete (slug, title, hex, categories, variants)

Related Issues

N/A

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new “Socratic AI” brand icon entry and corresponding SVG assets for default/light/dark variants.

Changes:

• Added a new socratic icon record to the icon registry (icons.json) with variant paths and metadata.
• Added default.svg, light.svg, and dark.svg assets under public/icons/socratic/.

Reviewed changes

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

File	Description
src/data/icons.json	Registers the new socratic icon and points to its SVG variants.
public/icons/socratic/light.svg	Adds the light variant SVG asset (currently very large/unoptimized).
public/icons/socratic/default.svg	Adds the default variant SVG asset (matches light variant content).
public/icons/socratic/dark.svg	Adds the dark variant SVG asset (different format/header than other variants).

Comments suppressed due to low confidence (2)

src/data/icons.json:1

• This file appears to be ordered by slug (the next entry is "01dotai"). Inserting "socratic" at the top breaks that ordering, making future merges and lookup harder; please place the new entry in the correct sorted position (or follow the project’s established ordering rule for this list).
  src/data/icons.json:1
• The variant naming looks inverted based on the actual fills: light.svg/default.svg contain many near-white fills (e.g., #FEFEFE), while dark.svg uses fill="#000000". If consumers interpret light as “for light mode/light background” and dark as “for dark mode/dark background”, this will produce low-contrast rendering. Consider swapping the light/dark assets (or recoloring them) so each variant has appropriate contrast for its intended background.

---

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

[greptile-apps]

Greptile Summary

This PR adds three SVG icon variants (default, light, dark) for Socratic AI along with a corresponding icons.json entry. All three files appear to have been produced by raster-to-vector tracing tools rather than being cleaned-up vector originals, leaving behind significant quality issues that need to be resolved before merging.

• dark.svg uses the deprecated SVG 1.0 format with a DOCTYPE declaration, pt units, and extra decimal places, making it inconsistent with the other two variants.
• default.svg and light.svg are byte-for-byte identical and each contain ~260 empty <path d=\"\"> elements — tracing artifacts that add no visual content but significantly bloat the files.
• The icons.json entry has a near-white hex value (fefefe), is inserted at position 0 rather than in alphabetical order, and carries a future dateAdded.

Confidence Score: 3/5

Not ready to merge — the SVG files have structural quality issues that will result in a malformed dark icon and duplicate default/light variants.

The dark icon was produced by a different tool than the other two and carries a deprecated SVG 1.0 format with pt units, which can cause sizing inconsistencies in rendering pipelines. The default and light SVGs are identical to each other and contain hundreds of empty path elements left over from bitmap tracing — users will receive no differentiation between those two variants. The brand hex color of near-white is atypical and unverified.

All three SVG files need attention: dark.svg requires reformatting to SVG 1.1 with pixel units, and default.svg / light.svg require removal of empty paths and differentiation from each other.

Important Files Changed

Filename	Overview
public/icons/socratic/dark.svg	SVG 1.0 format with deprecated DOCTYPE, pt units, and extra decimal places — inconsistent with the other two variants which use SVG 1.1 and unitless dimensions; appears generated by a different tool (potrace).
public/icons/socratic/default.svg	Contains ~260 empty <path d=""> elements (bitmap-tracing artifacts) that render nothing but bloat the file; the visual content is limited to 5 actual paths. Byte-for-byte identical to light.svg.
public/icons/socratic/light.svg	Identical content to default.svg — same empty-path bloat and same fill color (#FEFEFE), making light and default variants indistinguishable.
src/data/icons.json	New socratic entry has correct schema structure, but uses a near-white hex value (fefefe), is inserted at position 0 rather than alphabetical order, and has a future dateAdded.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[icons.json entry: socratic] --> B[default variant]
    A --> C[light variant]
    A --> D[dark variant]
    B --> E[default.svg
SVG 1.1 ✓
~260 empty paths ✗
fill=#FEFEFE]
    C --> F[light.svg
SVG 1.1 ✓
~260 empty paths ✗
Identical to default ✗
fill=#FEFEFE]
    D --> G[dark.svg
SVG 1.0 / DOCTYPE ✗
pt units ✗
fill=#000000]

Loading

_{Reviews (1): Last reviewed commit: "add socratic ai icons" | Re-trigger Greptile}

[greptile-apps]

Outdated SVG 1.0 format with pt units

dark.svg uses a deprecated SVG 1.0 DOCTYPE declaration and pt (point) units for width and height, while default.svg and light.svg both use SVG 1.1 with unitless pixel dimensions. The pt unit can cause inconsistent sizing across rendering contexts, and the external DTD reference (http://www.w3.org/TR/2001/REC-SVG-20010904/DTD/svg10.dtd) may trigger an unnecessary network fetch in some parsers. The viewBox values also include unnecessary decimal places (1024.000000 vs 1024). This file appears to have been generated by a different tool (likely potrace) compared to the other two variants.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

[greptile-apps]

Hundreds of empty <path> elements bloating the SVG

Lines 16–302 in both default.svg and light.svg contain ~260 empty path elements of the form <path d="" fill="..." transform="translate(0,0)"/>. Empty d attributes produce no visual output — these are vestigial artifacts from a bitmap-to-SVG conversion tool (the near-duplicate per-pixel fill colors confirm this was auto-traced from a raster image). They bloat the file, slow parse times, and make the SVG unmaintainable. These should be removed before merging. The light.svg file is byte-for-byte identical to default.svg, making the light variant indistinguishable from the default.

[greptile-apps]

Brand hex value is near-white (fefefe)

The hex field is set to fefefe, which is effectively white. This is an unusual brand color and may indicate the default icon color was used as the hex value instead of the actual Socratic AI brand color. Other entries in the file use meaningful brand colors (e.g., 003425 for 01.AI). If fefefe is correct per the official brand guidelines, a brief note in the PR description would help confirm it's intentional.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!