-
-
Notifications
You must be signed in to change notification settings - Fork 109
docs: normalize heading hierarchy in system components section #289
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -83,25 +83,25 @@ Ell-ena uses PostgreSQL (via Supabase) with strict Row-Level Security (RLS) and | |
|
|
||
| ## β¨ System Components | ||
|
|
||
| #### 1. Frontend Layer (Flutter) | ||
| ### 1. Frontend Layer (Flutter) | ||
| - **Auth Module**: Handles user authentication, team management, and role-based access control | ||
| - **Task Manager**: Processes task creation, updates, and workflow management | ||
| - **Meeting Manager**: Manages meeting scheduling, transcription, and contextual analysis | ||
| - **Chat Interface**: Provides natural language interaction with the AI assistant | ||
|
|
||
| #### 2. Supabase Service Layer | ||
| ### 2. Supabase Service Layer | ||
| - **Auth Client**: Manages authentication tokens and session state | ||
| - **Data Client**: Handles real-time data synchronization with PostgreSQL | ||
| - **Storage Client**: Manages file uploads and retrieval | ||
| - **RPC Client**: Executes remote procedure calls to Edge Functions | ||
|
|
||
|
Comment on lines
+92
to
97
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. π Maintainability & Code Quality | π‘ Minor | β‘ Quick win Add a blank line after this heading.
π§° Toolsπͺ markdownlint-cli2 (0.22.1)[warning] 92-92: Headings should be surrounded by blank lines (MD022, blanks-around-headings) π€ Prompt for AI AgentsSources: Path instructions, Linters/SAST tools |
||
| #### 3. Backend Layer (Supabase) | ||
| ### 3. Backend Layer (Supabase) | ||
| - **Authentication**: Handles user identity, security, and session management | ||
| - **PostgreSQL DB**: Stores structured data with Row-Level Security policies | ||
| - **Object Storage**: Manages binary assets like audio recordings and documents | ||
| - **Edge Functions**: Executes serverless functions for business logic | ||
|
|
||
|
Comment on lines
+98
to
103
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. π Maintainability & Code Quality | π‘ Minor | β‘ Quick win Add a blank line after this heading.
π§° Toolsπͺ markdownlint-cli2 (0.22.1)[warning] 98-98: Headings should be surrounded by blank lines (MD022, blanks-around-headings) π€ Prompt for AI AgentsSources: Path instructions, Linters/SAST tools |
||
| #### 4. AI Processing Pipeline | ||
| ### 4. AI Processing Pipeline | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. π Maintainability & Code Quality | π‘ Minor | β‘ Quick win Add a blank line after this heading.
π§° Toolsπͺ markdownlint-cli2 (0.22.1)[warning] 104-104: Headings should be surrounded by blank lines (MD022, blanks-around-headings) π€ Prompt for AI AgentsSources: Path instructions, Linters/SAST tools |
||
| - **NLU Processor**: Processes natural language using Gemini API | ||
| - **Vector Database**: Stores and retrieves semantic embeddings for context-aware searches | ||
| - **Embedding Generator**: Creates vector embeddings from text for semantic similarity | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π Maintainability & Code Quality | π‘ Minor | β‘ Quick win
Add a blank line after this heading.
MD022flags the heading as not being surrounded by blank lines; insert one blank line before the list for cleaner Markdown rendering. As per path instructions, Markdown files should have proper formatting and structure.π§° Tools
πͺ markdownlint-cli2 (0.22.1)
[warning] 86-86: Headings should be surrounded by blank lines
Expected: 1; Actual: 0; Below
(MD022, blanks-around-headings)
π€ Prompt for AI Agents
Sources: Path instructions, Linters/SAST tools