dataface 0.1.6.dev82__py3-none-any.whl → 0.1.6.dev129__py3-none-any.whl

dataface/agent_api/_paths.py

@@ -6,9 +6,8 @@ from dataclasses import dataclass
 from pathlib import Path
 
 from dataface.core.project_roots import (
-    discover_render_context as discover_render_context,
-    discovery_boundary_for_face as discovery_boundary_for_face,
     find_dft_root as find_dft_root,
+    find_project_root as find_project_root,
 )
 from dataface.core.scoped_paths import (
     resolve_scoped_path as resolve_scoped_path,
@@ -60,8 +59,8 @@ class FaceRenderContext:
     """Path resolution result from a face path + project root.
 
     Adapter registry is built by `ProjectSession.open`, not by the context — call sites
-    open a `ProjectSession` with `project_root` + `dbt_project_path` and read the
-    registry off `project.adapter_registry`.
+    open a `ProjectSession` with `project_root` and read the registry off
+    `project.adapter_registry`.
     """
 
     face_file: Path
@@ -69,18 +68,16 @@ class FaceRenderContext:
     scoped_base: Path
     project_root: Path
     output_dir: Path
-    dbt_project_path: Path | None = None
 
 
 def build_face_render_context(
     face_path: Path,
     project_dir: Path,
 ) -> FaceRenderContext:
-    """Resolve a face path and walk for dbt context.
+    """Resolve a face path against the given project root.
 
-    ``project_dir`` is authoritative; the walk only contributes the dbt project
-    path. Callers must resolve their project dir first (e.g. via
-    ``resolve_project_dir(raw_dir)`` at the CLI boundary).
+    ``project_dir`` is authoritative. Callers must resolve their project dir first
+    (e.g. via ``resolve_project_dir(raw_dir)`` at the CLI boundary).
     """
     if face_path.is_absolute():
         face_file = face_path.resolve()
@@ -89,10 +86,6 @@ def build_face_render_context(
     else:
         face_file = (project_dir / face_path).resolve()
 
-    _, dbt_project_path = discover_render_context(
-        face_file.parent,
-        discovery_boundary_for_face(face_file.parent, project_dir),
-    )
     project_root = project_dir
 
     try:
@@ -109,7 +102,6 @@ def build_face_render_context(
         scoped_base=project_root,
         project_root=project_root,
         output_dir=project_root,
-        dbt_project_path=dbt_project_path,
     )
 
 
@@ -122,21 +114,18 @@ class YamlRenderContext:
 
     project_root: Path
     output_dir: Path
-    dbt_project_path: Path | None = None
 
 
 def build_yaml_render_context(
     project_dir: Path,
 ) -> YamlRenderContext:
-    """Walk for dbt context anchored at the given project root.
+    """Resolve the project root for rendering inline YAML.
 
     ``project_dir`` is authoritative. Callers must resolve their project dir
     first (e.g. via ``resolve_project_dir(raw_dir)`` at the CLI boundary).
     """
     project_root = project_dir.resolve()
-    _, dbt_project_path = discover_render_context(project_root, None)
     return YamlRenderContext(
         project_root=project_root,
         output_dir=project_root,
-        dbt_project_path=dbt_project_path,
     )

dataface/agent_api/docs/yaml-reference.md

@@ -128,8 +128,8 @@ Authored patch for bar and histogram charts.
 | `sort` | [ChartSort](#chartsort) | ✓ | Sort configuration: field to sort by and direction (asc/desc). |
 | `labels` | [ChartLabels](#chartlabels) | ✓ | Per-row text annotations near each data anchor. |
 | `data_table` | [ChartDataTable](#chartdatatable) | ✓ | Optional mini data-grid attached below/above the chart. |
-| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
-| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
+| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
+| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
 | `style` | [BarChartStyle](#barchartstyle) | ✓ | Chart-local style overrides. |
 
 <a id="linechart"></a>
@@ -156,8 +156,8 @@ Authored patch for line charts.
 | `sort` | [ChartSort](#chartsort) | ✓ | Sort configuration: field to sort by and direction (asc/desc). |
 | `labels` | [ChartLabels](#chartlabels) | ✓ | Per-row text annotations near each data anchor. |
 | `data_table` | [ChartDataTable](#chartdatatable) | ✓ | Optional mini data-grid attached below/above the chart. |
-| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
-| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
+| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
+| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
 | `style` | [LineChartStyle](#linechartstyle) | ✓ | Chart-local style overrides. |
 
 <a id="areachart"></a>
@@ -184,8 +184,8 @@ Authored patch for area charts.
 | `sort` | [ChartSort](#chartsort) | ✓ | Sort configuration: field to sort by and direction (asc/desc). |
 | `labels` | [ChartLabels](#chartlabels) | ✓ | Per-row text annotations near each data anchor. |
 | `data_table` | [ChartDataTable](#chartdatatable) | ✓ | Optional mini data-grid attached below/above the chart. |
-| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
-| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
+| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
+| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
 | `style` | [AreaChartStyle](#areachartstyle) | ✓ | Chart-local style overrides. |
 
 <a id="scatterchart"></a>
@@ -212,8 +212,8 @@ Authored patch for scatter charts.
 | `sort` | [ChartSort](#chartsort) | ✓ | Sort configuration: field to sort by and direction (asc/desc). |
 | `labels` | [ChartLabels](#chartlabels) | ✓ | Per-row text annotations near each data anchor. |
 | `data_table` | [ChartDataTable](#chartdatatable) | ✓ | Optional mini data-grid attached below/above the chart. |
-| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
-| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
+| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
+| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
 | `size` | str | ✓ | Field used to size-encode data points (quantitative). |
 | `shape` | str | ✓ | Field used to shape-encode data points (categorical). |
 | `style` | [ScatterChartStyle](#scatterchartstyle) | ✓ | Chart-local style overrides. |
@@ -242,8 +242,8 @@ Authored patch for heatmap charts.
 | `sort` | [ChartSort](#chartsort) | ✓ | Sort configuration: field to sort by and direction (asc/desc). |
 | `labels` | [ChartLabels](#chartlabels) | ✓ | Per-row text annotations near each data anchor. |
 | `data_table` | [ChartDataTable](#chartdatatable) | ✓ | Optional mini data-grid attached below/above the chart. |
-| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
-| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Not valid on kpi, table, callout, or spark_bar — those renderers own their own sizing contracts. |
+| `height` | int \| float | ✓ | Explicit chart height in pixels. Positive number only. When set, overrides aspect_ratio and theme cascade. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
+| `width` | int \| float | ✓ | Chart width hint in pixels. Positive number only. Used by the label-overlap heuristic to determine whether x-axis labels need tilting. Does not override the layout slot width — the chart still fills its allocated container. Only valid on cartesian chart families: area, bar, heatmap, histogram, line, and scatter. Other chart families use renderer-owned or layout-owned sizing contracts. |
 | `style` | [HeatmapChartStyle](#heatmapchartstyle) | ✓ | Chart-local style overrides. |
 
 <a id="piechart"></a>
@@ -2372,10 +2372,11 @@ Authored overlay for SparkAreaStyle.
 
 <a id="geoshapemarkstyle"></a>
 ## GeoshapeMarkStyle
-Authored overlay for GeoshapeMarkStyle. Geoshape (choropleth) mark boundary stroke.
+Authored overlay for GeoshapeMarkStyle. Geoshape (choropleth) mark fill and boundary stroke.
 
 | Field | Type | Optional | Description |
 |-------|------|:--------:|-------------|
+| `fill` | str | ✓ | Neutral geoshape fill color. |
 | `stroke` | [StrokeStyle](#strokestyle) | ✓ | Geoshape boundary stroke style. |
 
 <a id="circlemarkstyle"></a>

dataface/agent_api/project_session.py

@@ -17,7 +17,6 @@ else:
 # Verb-forwarder imports use SUBMODULES so test monkeypatches on the
 # module attribute affect the call site.
 from dataface.agent_api import (
-    dashboards as _dashboards,
     describe as _describe,
     describe_query as _describe_query,
     query as _query,
@@ -42,7 +41,7 @@ from dataface.core.inspect.query_validator import (
     validate_query as _core_validate_query,
 )
 from dataface.core.project import Project
-from dataface.core.project_roots import discover_render_context
+from dataface.core.project_roots import find_project_root
 
 if TYPE_CHECKING:
     from dataface.core.dashboard import RenderedDashboard, RenderFormat
@@ -71,7 +70,6 @@ class ProjectSession:
     cache: DuckDBCache | None
     _owns_registry: bool
     _read_only: bool
-    _dbt_project_path: Path | None
     _connection_string: str | None
     _dialect: str
     _target: str
@@ -96,7 +94,6 @@ class ProjectSession:
             # short-circuit the build on first access.
             self.__dict__["adapter_registry"] = adapter_registry
         self._read_only = read_only
-        self._dbt_project_path = None
         self._connection_string = None
         self._dialect = "duckdb"
         self._target = "dev"
@@ -111,7 +108,6 @@ class ProjectSession:
         *,
         cache: DuckDBCache | None = None,
         read_only: bool = True,
-        dbt_project_path: Path | None = None,
         connection_string: str | None = None,
         dialect: str = "duckdb",
         target: str = "dev",
@@ -135,7 +131,6 @@ class ProjectSession:
             cache=cache,
             read_only=read_only,
         )
-        session._dbt_project_path = dbt_project_path
         session._connection_string = connection_string
         session._dialect = dialect
         session._target = target
@@ -151,7 +146,6 @@ class ProjectSession:
         *,
         cache: DuckDBCache | None = None,
         read_only: bool = True,
-        dbt_project_path: Path | None = None,
         connection_string: str | None = None,
         dialect: str = "duckdb",
         target: str = "dev",
@@ -166,7 +160,6 @@ class ProjectSession:
         this method stores it directly without re-wrapping.
         """
         session = cls(project=project, cache=cache, read_only=read_only)
-        session._dbt_project_path = dbt_project_path
         session._connection_string = connection_string
         session._dialect = dialect
         session._target = target
@@ -185,13 +178,8 @@ class ProjectSession:
     ) -> Self:
         """Open a ProjectSession rooted at the project directory discovered upward from face_file."""
         resolved = Path(face_file).resolve()
-        project_root, dbt_project_path = discover_render_context(resolved.parent, None)
-        return cls.open(
-            project_root,
-            read_only=read_only,
-            cache=cache,
-            dbt_project_path=dbt_project_path,
-        )
+        project_root = find_project_root(resolved.parent, boundary=None)
+        return cls.open(project_root, read_only=read_only, cache=cache)
 
     def __enter__(self) -> Self:
         return self
@@ -210,7 +198,6 @@ class ProjectSession:
         return build_adapter_registry(
             self.project,
             read_only=self._read_only,
-            dbt_project_path=self._dbt_project_path,
             connection_string=self._connection_string,
             profile_type=self._dialect,
             target=self._target,
@@ -279,11 +266,6 @@ class ProjectSession:
 
     # ── Verb forwarders ──────────────────────────────────────────────────────
 
-    def validate(self, face_path: Path) -> ValidateResult:
-        result = _validate.validate(face_path, project=self.project)
-        annotated = _validate.annotate_with_data_lint([result], project_session=self)
-        return annotated[0]
-
     def validate_paths(self, paths: list[Path] | None) -> list[ValidateResult]:
         results = _validate.validate_paths(paths, project=self.project)
         return _validate.annotate_with_data_lint(results, project_session=self)
@@ -346,32 +328,9 @@ class ProjectSession:
             relationship_context=self._relationship_context,
         )
 
-    def describe_face(self, path: Path) -> _describe.DescribeFaceResult:
-        return _describe.describe_face(path, project=self.project)
-
     def describe_paths(self, paths: list[Path]) -> list[_describe.DescribeFaceResult]:
         return _describe.describe_paths(paths, project=self.project)
 
-    def list_dashboards(
-        self,
-        directory: Path | None = None,
-        recursive: bool = True,
-    ) -> _dashboards.ListDashboardsResult:
-        """When ``directory`` is None, defaults to ``self.project.root``. When provided, callers are responsible for passing an absolute path; relative paths resolve against the process working directory."""
-        return _dashboards.list_dashboards(
-            directory=directory if directory is not None else self.project.root,
-            recursive=recursive,
-        )
-
-    def get_dashboard(
-        self,
-        path: Path,
-        include_raw: bool = False,
-    ) -> _dashboards.CompiledDashboard:
-        return _dashboards.get_dashboard(
-            path, include_raw=include_raw, project=self.project
-        )
-
     def lookup_face_query_sql(
         self,
         name: str,

dataface/agent_api/render_face.py

@@ -15,10 +15,6 @@ from dataface.core.compile.markdown import (
 )
 from dataface.core.execute.duckdb_cache import DuckDBCache
 from dataface.core.project import Project
-from dataface.core.project_roots import (
-    discover_render_context,
-    discovery_boundary_for_face,
-)
 from dataface.core.render.face_api import compile_and_render
 
 
@@ -72,16 +68,7 @@ def render_face(
     if project is not None:
         if warnings_ignore is None:
             warnings_ignore = project.warnings_ignore
-        # Discover dbt_project.yml between the face and the project root.
-        # build_adapter_registry's own fallback would only walk upward from
-        # project.root, missing nested dbt projects under it.
-        _, dbt_project_path = discover_render_context(
-            face_file.parent,
-            discovery_boundary_for_face(face_file.parent, project.root),
-        )
-        project_session = ProjectSession.open(
-            project.root, cache=cache, dbt_project_path=dbt_project_path
-        )
+        project_session = ProjectSession.open(project.root, cache=cache)
     else:
         if warnings_ignore is None:
             warnings_ignore = frozenset()

dataface/ai/context.py

@@ -21,6 +21,7 @@ class DatafaceAIContext:
     # `dft chat` after binding; consumed by `_view_url` to build clickable
     # localhost URLs in `render_dashboard` responses (including as_link=True).
     server_port: int | None = None
+    prompt_context: dict[str, str] | None = None
 
     def resolve_dashboard_path(self, path: Path) -> Path:
         """Resolve a dashboard path inside the scoped dashboards directory.

dataface/cli/commands/render.py

@@ -129,11 +129,7 @@ def render_command(
 
     with (
         cache_from_env() as cache,
-        ProjectSession.open(
-            ctx.project_root,
-            cache=cache,
-            dbt_project_path=ctx.dbt_project_path,
-        ) as project_session,
+        ProjectSession.open(ctx.project_root, cache=cache) as project_session,
     ):
         result = project_session.render_dashboard(
             path=ctx.scoped_path,
@@ -219,11 +215,7 @@ def render_command_from_yaml(
 
     with (
         cache_from_env() as cache,
-        ProjectSession.open(
-            ctx.project_root,
-            cache=cache,
-            dbt_project_path=ctx.dbt_project_path,
-        ) as project_session,
+        ProjectSession.open(ctx.project_root, cache=cache) as project_session,
     ):
         result = project_session.render_dashboard(
             yaml_content=yaml_content,

dataface/core/compile/compiler.py

@@ -271,9 +271,7 @@ def compile(
         yaml_content: YAML string to compile
         options: Optional compilation options
         base_dir: Base directory for resolving file references
-        project_sources: Project-level sources, ready to use. Callers that
-            hold a ``ProjectSession`` thread ``project_session.sources``;
-            one-shot callers pre-load via ``load_project_sources(Project(project_dir))``.
+        project_sources: Project-level sources.
 
     Returns:
         CompileResult with compiled face or errors
@@ -440,6 +438,7 @@ def compile_file(
     apply_meta: bool = True,
     *,
     project: Project,
+    markdown_metadata_table: bool = False,
 ) -> CompileResult:
     """Compile a YAML file to a Face.
 
@@ -451,6 +450,8 @@ def compile_file(
         options: Optional compilation options
         apply_meta: If True, resolve and apply meta.yaml chain (default: True)
         project: Project for meta resolution and source loading
+        markdown_metadata_table: When True and file_path is a .md, prepend
+            non-face frontmatter keys as a metadata table before the body.
 
     Returns:
         CompileResult with compiled face or errors
@@ -471,13 +472,14 @@ def compile_file(
         )
 
     # Markdown report files: translate to YAML before compiling
-    if file_path.suffix.lower() == ".md":
-        from dataface.core.compile.markdown import (
-            MARKDOWN_NOT_FACE_MESSAGE,
-            is_markdown_face,
-            markdown_to_yaml,
-        )
+    from dataface.core.compile.markdown import (
+        MARKDOWN_NOT_FACE_MESSAGE,
+        MARKDOWN_SUFFIXES,
+        is_markdown_face,
+        markdown_to_yaml,
+    )
 
+    if file_path.suffix.lower() in MARKDOWN_SUFFIXES:
         if not is_markdown_face(file_path):
             return CompileResult(
                 errors=[CompilationError(MARKDOWN_NOT_FACE_MESSAGE).to_structured()]
@@ -485,7 +487,9 @@ def compile_file(
 
         try:
             raw_text = file_path.read_text(encoding="utf-8")
-            yaml_content = markdown_to_yaml(raw_text)
+            yaml_content = markdown_to_yaml(
+                raw_text, metadata_table=markdown_metadata_table
+            )
         except (OSError, ValueError) as e:
             return CompileResult(
                 errors=[CompilationError(f"Markdown parse error: {e}").to_structured()]

dataface/core/compile/config.py

@@ -200,6 +200,11 @@ def get_project_server_config(project: Project) -> ServerConfig:
     return ServerConfig.model_validate(_deep_merge(base, server_section))
 
 
+def get_project_markdown_metadata_table(project: Project) -> bool:
+    """Return whether the project has the markdown_metadata_table flag enabled."""
+    return get_project_server_config(project).markdown_metadata_table
+
+
 def load_config(path: Path) -> Config:
     """Load configuration from a custom YAML file.
 
@@ -610,15 +615,15 @@ def load_project_sources(project: Project) -> ProjectSourcesConfig:
             config.default = yml_default
 
     # Absolutize paths now that all sources are merged
-    config.sources = _absolutize_source_paths(config.sources, project.root)
+    config.sources = _absolutize_source_paths(config.sources, project)
     return config
 
 
 def _absolutize_source_paths(
     sources: dict[str, dict[str, Any]],
-    project_dir: Path,
+    project: Project,
 ) -> dict[str, dict[str, Any]]:
-    """Absolutize relative file/database paths against ``project_dir``."""
+    """Absolutize relative file/database paths against ``project.root``."""
     resolved: dict[str, dict[str, Any]] = {}
     for name, config in sources.items():
         cfg = dict(config)
@@ -630,11 +635,11 @@ def _absolutize_source_paths(
                 and path_value != ":memory:"
                 and not Path(path_value).is_absolute()
             ):
-                cfg["path"] = str((project_dir / path_value).resolve())
+                cfg["path"] = str((project.root / path_value).resolve())
         elif source_type in {"csv", "parquet", "json"}:
             file_value = cfg.get("file")
             if isinstance(file_value, str) and not Path(file_value).is_absolute():
-                cfg["file"] = str((project_dir / file_value).resolve())
+                cfg["file"] = str((project.root / file_value).resolve())
         resolved[name] = cfg
     return resolved
 

dataface/core/compile/detect.py

@@ -16,9 +16,21 @@ Detection methods (in order of precedence):
 4. Markdown reports: .md files with Dataface YAML frontmatter
 """
 
+import os
 import re
 from pathlib import Path
 
+_SKIP_SCAN_DIRS = {
+    ".git",
+    ".venv",
+    "__pycache__",
+    ".mypy_cache",
+    ".pytest_cache",
+    "build",
+    "dist",
+    "node_modules",
+}
+
 
 def is_dataface_file(path: Path | str) -> bool:
     """
@@ -33,9 +45,9 @@ def is_dataface_file(path: Path | str) -> bool:
     path = Path(path)
 
     # Markdown report files
-    if path.suffix.lower() == ".md":
-        from dataface.core.compile.markdown import is_markdown_face
+    from dataface.core.compile.markdown import MARKDOWN_SUFFIXES, is_markdown_face
 
+    if path.suffix.lower() in MARKDOWN_SUFFIXES:
         return is_markdown_face(path)
 
     # Must be a YAML file
@@ -103,8 +115,22 @@ def find_dataface_files(directory: Path | str, recursive: bool = True) -> list[P
         List of paths to Dataface files
     """
     directory = Path(directory)
-    yaml_pattern = "**/*.y*ml" if recursive else "*.y*ml"
-    md_pattern = "**/*.md" if recursive else "*.md"
-
-    candidates = list(directory.glob(yaml_pattern)) + list(directory.glob(md_pattern))
+    face_suffixes = (".yml", ".yaml", ".md", ".markdown")
+
+    if recursive:
+        candidates: list[Path] = []
+        for root, dirs, files in os.walk(directory, topdown=True):
+            dirs[:] = [d for d in dirs if d not in _SKIP_SCAN_DIRS]
+            root_path = Path(root)
+            candidates.extend(
+                root_path / name for name in files if name.endswith(face_suffixes)
+            )
+    else:
+        candidates = [
+            p
+            for p in directory.iterdir()
+            if p.is_file() and p.name.endswith(face_suffixes)
+        ]
+
+    candidates = sorted(candidates)
     return [p for p in candidates if is_dataface_file(p)]