Skip to content

Copilot comments#235

Open
bourgeoa wants to merge 2 commits intomainfrom
longchat-shapes
Open

Copilot comments#235
bourgeoa wants to merge 2 commits intomainfrom
longchat-shapes

Conversation

@bourgeoa
Copy link
Copy Markdown
Contributor

@bourgeoa bourgeoa commented Apr 8, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 8, 2026 16:26
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates Long Chat SHACL shapes and example Turtle data to improve validation correctness and schema.org namespace compatibility.

Changes:

  • Tighten ui:backgroundColor validation to a fully-anchored 6-digit hex color regex.
  • Allow a sioc:Thread to be referenced by multiple messages via sioc:has_reply (e.g., replacements pointing at the same thread).
  • Switch the example’s schema: prefix to https://schema.org/ and add an explicit schema_http: prefix.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
shapes/longchat-shapes.ttl Improves SHACL constraints (hex color regex; thread ↔ message cardinality).
shapes/longchat-example.ttl Updates schema.org prefixes to align with dual http/https compatibility used by the shapes.

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

@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix schema: <http://schema.org/> .
@prefix schema: <https://schema.org/> .
@prefix schema_http: <http://schema.org/> .
Copy link

Copilot AI Apr 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

schema_http prefix is introduced but not used anywhere in this example file. Consider removing the unused prefix, or add an example triple that uses the schema_http: namespace (if the intent is to demonstrate compatibility with both schema.org IRIs).

Suggested change
@prefix schema_http: <http://schema.org/> .

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants