refactor(gfql): keep schema accessor minimal

Author: lmeyerov

CHANGELOG.md

@@ -10,7 +10,7 @@ This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
 
 ### Added
 - **GFQL schema effects (#1485)**: Added an internal typed schema-effect model for graph-growing GFQL calls so bound experimental `GraphSchema` snapshots are updated after successful degree, PageRank-style node-property writes, and edge-property write calls. Later local validation can see properties added by those calls without exposing a public `SchemaEffect` API or changing remote GFQL transport.
-- **GFQL schema accessor (#1632)**: Added experimental read-only `g.schema` and `g.has_schema()` accessors for inspecting a `GraphSchema` bound through `bind(schema=...)` without reaching into private `_gfql_schema` storage.
+- **GFQL schema accessor (#1632)**: Added experimental read-only `g.schema` for inspecting a `GraphSchema` bound through `bind(schema=...)` without reaching into private `_gfql_schema` storage.
 - **GFQL NetworkX CALL parity (#1058)**: Expanded the local Cypher `graphistry.nx.*` CALL surface with explicit NetworkX dispatch for `degree_centrality`, `closeness_centrality`, `eigenvector_centrality`, `katz_centrality`, `connected_components`, `strongly_connected_components`, `core_number`, and multi-output `hits`, including row and `.write()` coverage.
 - **NetworkX/SciPy optional dependency policy (#1618)**: Declared supported `networkx>=2.5,<4` and optional `scipy>=1.5,<2` ranges for NetworkX-backed GFQL CALL procedures, with runtime version guards and a focused lower/current-upper CI matrix.
 - **GFQL schema Arrow boundary APIs (#1339)**: Added experimental public schema↔Arrow import/export helpers, graph-level Arrow declaration payloads, and opt-in `schema_validate='strict'|'autofix'` enforcement for `plot()`, `upload()`, `to_arrow()`, and `validate_arrow_schema()` when a `GraphSchema` is bound.

docs/source/api/plotter.rst

@@ -28,10 +28,10 @@ declared Arrow types after normal Arrow conversion. The default
 ``schema_validate=False`` preserves existing behavior.
 
 Use the experimental read-only ``g.schema`` accessor to inspect the bound
-``GraphSchema`` object, or ``g.has_schema()`` when only a predicate is needed.
-This reports only the local declaration attached through ``bind(schema=...)``:
-it does not infer a schema from data, fetch a remote dataset schema, or serialize
-the schema into ``gfql_remote()`` requests.
+``GraphSchema`` object. Check ``g.schema is not None`` when only a predicate is
+needed. This reports only the local declaration attached through
+``bind(schema=...)``: it does not infer a schema from data, fetch a remote
+dataset schema, or serialize the schema into ``gfql_remote()`` requests.
 
 .. toctree::
    :maxdepth: 3

docs/source/gfql/schema.rst

@@ -74,7 +74,7 @@ check against.
 
    g.gfql_validate("MATCH (p:Person)-[:WORKS_AT]->(c:Company) RETURN p.name")
    assert g.schema is schema
-   assert g.has_schema()
+   assert g.schema is not None
 
 Schema Objects
 --------------
@@ -98,10 +98,11 @@ Schema Objects
   makes schema-bound ``g.gfql_validate(...)`` permissive by default; callers can
   still override per call with ``g.gfql_validate(..., strict=True)``.
 
-``g.schema`` and ``g.has_schema()``
+``g.schema``
   Read back the experimental ``GraphSchema`` bound with ``bind(schema=...)``.
-  ``g.schema`` returns the bound object or ``None``; ``g.has_schema()`` returns
-  a matching boolean. Use ``bind(schema=...)`` to attach schemas, not assignment.
+  ``g.schema`` returns the bound object or ``None``. Use
+  ``g.schema is not None`` when only a predicate is needed. Use
+  ``bind(schema=...)`` to attach schemas, not assignment.
   This is local declaration introspection only. It does not infer schemas from
   data, fetch or hydrate remote dataset schemas, or serialize schemas into
   ``gfql_remote()`` requests in this release.

graphistry/Plottable.py

@@ -376,9 +376,6 @@ def copy(self) -> 'Plottable':
     def schema(self) -> Optional["GraphSchema"]:
         ...
 
-    def has_schema(self) -> bool:
-        ...
-
     # ### ComputeMixin
 
     def get_indegrees(self, col: str = 'degree_in') -> 'Plottable':

graphistry/PlotterBase.py

@@ -1806,10 +1806,6 @@ def schema(self) -> Optional["GraphSchema"]:
         """
         return self._gfql_schema
 
-    def has_schema(self) -> bool:
-        """Return ``True`` when this plotter has a local GFQL schema bound."""
-        return self._gfql_schema is not None
-
     def copy(self) -> Plottable:
         return copy.copy(self)
 

graphistry/tests/compute/gfql/test_public_schema.py

@@ -191,7 +191,6 @@ def test_bind_schema_is_chainable_and_used_by_preflight() -> None:
 
     assert g._gfql_schema is schema
     assert g.schema is schema
-    assert g.has_schema() is True
     report = g.gfql_validate("MATCH (p:Person)-[:WORKS_AT]->(c:Company) RETURN p.name AS name")
     assert report["ok"] is True
 
@@ -201,7 +200,6 @@ def test_schema_accessor_returns_bound_schema() -> None:
     g = _graph(schema)
 
     assert g.schema is schema
-    assert g.has_schema() is True
 
 
 def test_schema_accessor_is_read_only() -> None:
@@ -218,7 +216,6 @@ def test_schema_accessor_returns_none_when_unbound() -> None:
     g = graphistry.bind()
 
     assert g.schema is None
-    assert g.has_schema() is False
 
 
 def test_bound_schema_arrow_boundary_strict_passes() -> None: