Skip to content

docs: update problem statement with progressive discovery details - #14

Merged
chughtapan merged 11 commits into
mainfrom
docs/problem-statement
Oct 5, 2026
Merged

chughtapan merged 11 commits into
mainfrom
docs/problem-statement

Conversation

@kurtisvg

Copy link
Copy Markdown
Contributor

Initial strawman for a problem statement from @SamMorrowDrums, @helloeve, and myself.

Comment thread docs/problem-statement.md
Comment thread docs/problem-statement.md Outdated
Comment thread docs/problem-statement.md Outdated
cliffhall
cliffhall previously approved these changes Sep 7, 2026

@cliffhall cliffhall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 👍

@chughtapan

Copy link
Copy Markdown
Contributor

@copilot rebase on main

Copilot AI commented Sep 7, 2026

Copy link
Copy Markdown

@copilot rebase on main

Merged the latest main history into this branch in commit a36ff63. I used a merge commit rather than rebasing because the publishing workflow does not support force-pushing rewritten history.

chughtapan
chughtapan previously approved these changes Sep 7, 2026

@SamMorrowDrums SamMorrowDrums left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Some comments and suggestions, most weakly held. I am just struggling to get to the core of comparing things that are fundamentally different, in a space where many players do not seem to see any meaningful distinction, especially been unstructured and structured tool calling, and the consequences of either path.

🤷

Comment thread docs/problem-statement.md Outdated
Comment thread docs/problem-statement.md
Comment thread docs/problem-statement.md
Comment thread docs/problem-statement.md Outdated
Comment thread docs/problem-statement.md Outdated
@kurtisvg
kurtisvg dismissed stale reviews from chughtapan and cliffhall via 3a930aa September 28, 2026 02:49
Copilot AI lite review requested due to automatic review settings September 28, 2026 02:49
@kurtisvg

Copy link
Copy Markdown
Contributor Author

Addressed the review feedback:

  • Made naming disambiguation a non-goal (thread).
  • Separated discovery correctness from prompt-cache efficiency (thread).
  • Preserved authorization/cache-scope semantics and recorded auditability as an open question (thread).
  • Clarified the CLI/Skills comparison focuses on progressive loading (thread).
  • Removed the speculative implementation example (thread).

Also tightened the protocol rationale, evidence, scope, and evaluation criteria.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

A moderate authorization and configuration-context requirement remains unresolved.

Review effort: Lite
Findings: 2 Low severity

Open (2)
What changed in this PR

Updates the progressive discovery problem statement and adds an auditability question.

Changes:

  • Documents motivation, examples, requirements, and evaluation criteria.
  • Adds discovery auditability to the open questions.
File Description
docs/​problem-statement.md Expands the progressive discovery problem statement and requirements.
docs/​open-questions.md Adds a discovery auditability question.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/problem-statement.md
[Related Work](related-work.md) |
[Contributing](../CONTRIBUTING.md)
# Progressive Discovery Problem Statement

Comment thread docs/problem-statement.md
Comment on lines +129 to +131
bundled in plugins or “connectors” should be capable of exposing their full
surface, currently the lack of user configuration capabilities frequently
prevents this.
@kurtisvg
kurtisvg force-pushed the docs/problem-statement branch from 3a930aa to ae7054d Compare September 28, 2026 02:55
@chughtapan
chughtapan merged commit 9529428 into main Oct 5, 2026
4 checks passed
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.

8 participants