Merge pull request #6 from nic01asFr/claude/fix-grist-ctx-parameter-Xqhnr

Fix table and column creation
Add schema tools
Fix files download

Author: nic01asFr

CHANGELOG.md

@@ -0,0 +1,104 @@
+# Changelog
+
+Tous les changements notables de ce projet seront documentés dans ce fichier.
+
+Le format est basé sur [Keep a Changelog](https://keepachangelog.com/fr/1.0.0/),
+et ce projet adhère au [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [0.2.0] - 2024-12-30
+
+### Ajouté
+
+- **Nouvel outil `create_schema`** : Création de schémas complets en une opération
+  - Création séquentielle : tables simples → résolution refs → colonnes référence
+  - Support des relations inverses (`reverse` dans la définition)
+  - Insertion optionnelle de données initiales
+  - Rapport détaillé avec warnings et erreurs
+
+- **Nouvel outil `create_reference_column`** : Création de références simplifiée
+  - Résolution automatique de `visibleCol` par nom de colonne
+  - Validation de l'existence de la table cible
+  - Création optionnelle de la relation inverse
+
+- **Nouvel outil `validate_schema`** : Validation sans création
+  - Vérification des tables cibles
+  - Détection des colonnes Choice sans choix
+  - Liste des erreurs et warnings
+
+- **Nouvel outil `export_schema`** : Export de schémas existants
+  - Formats : JSON, YAML, Mermaid (diagramme ER)
+  - Option pour inclure les données
+
+- **Nouveau module `types.py`** : Définitions de types
+  - `ColumnDefinition`, `TableDefinition`, `SchemaDefinition`
+  - Mapping `GRIST_COLUMN_TYPES` avec validations
+  - Fonction `parse_schema_dict` pour parser les schémas
+
+- **Nouveau module `utils.py`** : Fonctions utilitaires
+  - `prepare_table_payload` : Format correct pour création de tables
+  - `prepare_column_payload` : Format correct pour colonnes
+  - `resolve_visible_col` : Résolution nom → colRef
+  - `check_table_exists` : Vérification d'existence
+
+### Corrigé
+
+- **`create_table`** : Format du payload corrigé
+  - Avant : `{"tables": [{"tableId": "X", "columns": [...]}]}`
+  - Après : `{"tables": [{"id": "X", "columns": [{"id": "col", "fields": {...}}]}]}`
+
+- **`modify_table`** : Format du payload corrigé
+  - Avant : `{"tables": [{"tableId": "X", "newTableId": "Y"}]}`
+  - Après : `{"tables": [{"id": "X", "fields": {"tableId": "Y"}}]}`
+
+- **`create_column` / `modify_column`** : `widgetOptions` sérialisé en JSON string
+  - L'API Grist attend une string, pas un dict
+  - Correction appliquée via `json.dumps()`
+
+- **`create_column`** : Support des `choices` pour `Choice`/`ChoiceList`
+  - Nouveau paramètre `choices: List[str]`
+  - Intégré automatiquement dans `widgetOptions`
+
+- **`download_table_csv`** : Implémentation complète
+  - Paramètre `tableId` correctement passé
+
+- **`download_document_excel`** : Gestion du paramètre obligatoire `tableId`
+  - Récupère automatiquement la première table si non spécifié
+
+### Modifié
+
+- **`create_column`** : Nouveaux paramètres
+  - `visible_col: int` : Pour les références
+  - `untie_col_id_from_label: bool` : Défaut True
+  - `description: str` : Description de la colonne
+  - `choices: List[str]` : Pour Choice/ChoiceList
+
+- **`modify_column`** : Nouveaux paramètres
+  - `visible_col: int`
+  - `untie_col_id_from_label: bool`
+  - `description: str`
+
+### Structure du projet
+
+```
+src/mcp_server_grist/
+├── types.py              # NOUVEAU: Types et validations
+├── tools/
+│   ├── __init__.py       # MODIFIÉ: Exports
+│   ├── utils.py          # NOUVEAU: Utilitaires
+│   ├── administration.py # MODIFIÉ: Corrections
+│   ├── schema.py         # NOUVEAU: Outils schéma
+│   └── downloads.py      # MODIFIÉ: Corrections
+└── ...
+```
+
+## [0.1.x] - Versions précédentes
+
+### Fonctionnalités initiales
+
+- Navigation : organisations, workspaces, documents, tables, colonnes
+- CRUD tables et colonnes
+- CRUD enregistrements
+- Requêtes SQL
+- Gestion des accès
+- Webhooks
+- Pièces jointes

mcp-server-grist-claude-fix-grist-ctx-parameter-Xqhnr.tar

@@ -408,13 +408,15 @@ async def download_doc(self, doc_id: str, nohistory: bool = False, template: boo
             response.raise_for_status()
             return response.content
 
-    async def download_doc_xlsx(self, doc_id: str, header: str = "label") -> bytes:
+    async def download_doc_xlsx(self, doc_id: str, table_id: Optional[str] = None, header: str = "label") -> bytes:
         """Télécharge un document au format Excel."""
         params = {"header": header}
+        if table_id:
+            params["tableId"] = table_id
 
-        logger.debug(f"Downloading document {doc_id} as Excel")
+        logger.debug(f"Downloading document {doc_id} as Excel with params: {params}")
         try:
-            async with httpx.AsyncClient(timeout=120.0) as client:  # Augmenter timeout
+            async with httpx.AsyncClient(timeout=120.0) as client:
                 response = await client.request(
                     method="GET",
                     url=f"{self.api_url.rstrip('/')}/docs/{doc_id}/download/xlsx",
@@ -423,17 +425,15 @@ async def download_doc_xlsx(self, doc_id: str, header: str = "label") -> bytes:
                 )
 
                 logger.debug(f"Excel download response status: {response.status_code}")
-                logger.debug(f"Excel download response headers: {dict(response.headers)}")
 
                 response.raise_for_status()
                 return response.content
 
         except httpx.TimeoutException as e:
             logger.error(f"Excel download timeout for doc {doc_id}: {e}")
-            raise ValueError(f"Excel download timeout - document may be too large. Try download_document_sqlite as alternative.")
+            raise ValueError(f"Excel download timeout - document may be too large.")
         except httpx.HTTPStatusError as e:
             logger.error(f"Excel download HTTP error for doc {doc_id}: {e}")
-            logger.error(f"Response text: {e.response.text}")
             raise ValueError(f"Excel download failed: {e.response.status_code} - {e.response.text}")
         except Exception as e:
             logger.error(f"Excel download unexpected error for doc {doc_id}: {e}")

mcp-server-grist-v0.2.0.tar.gz

@@ -3,6 +3,8 @@
 
 Ce module regroupe tous les outils MCP disponibles pour interagir
 avec l'API Grist, organisés par catégories fonctionnelles.
+
+v0.2.0: Ajout des outils de schéma (create_schema, create_reference_column, etc.)
 """
 
 from .access import register_access_tools
@@ -12,39 +14,43 @@
 from .navigation import register_navigation_tools
 from .queries import register_query_tools
 from .records import register_record_tools
+from .schema import register_schema_tools
 from .webhooks import register_webhook_tools
 
 
 def register_all_tools(mcp_server):
     """
     Enregistre tous les outils MCP disponibles sur le serveur.
-    
+
     Cette fonction centralise l'enregistrement de tous les outils
     pour faciliter leur gestion et leur découverte.
-    
+
     Args:
         mcp_server: L'instance du serveur MCP sur laquelle enregistrer les outils.
     """
     # Navigation (lecture)
     register_navigation_tools(mcp_server)
-    
+
     # Records (écriture)
     register_record_tools(mcp_server)
-    
+
     # Queries (requêtes SQL)
     register_query_tools(mcp_server)
-    
+
     # Administration
     register_admin_tools(mcp_server)
-    
+
     # Gestion des accès
     register_access_tools(mcp_server)
-    
+
     # Export et téléchargement
     register_export_tools(mcp_server)
-    
+
     # Pièces jointes
     register_attachment_tools(mcp_server)
-    
+
     # Webhooks
-    register_webhook_tools(mcp_server)
+    register_webhook_tools(mcp_server)
+
+    # Schéma (v0.2.0) - Outils de création de schéma
+    register_schema_tools(mcp_server)