Skip to content

Merge pull request #2 from inline-arc/testing#3

Open
inline-arc wants to merge 14 commits into
testingfrom
master
Open

Merge pull request #2 from inline-arc/testing#3
inline-arc wants to merge 14 commits into
testingfrom
master

Conversation

@inline-arc

Copy link
Copy Markdown
Owner

memory

Copilot AI review requested due to automatic review settings June 25, 2026 18:50

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 wasn't able to review any files in this pull request.


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

@inline-arc inline-arc requested a review from Copilot June 26, 2026 11:43

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 wasn't able to review any files in this pull request.

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai

@agentwaspai

agentwaspai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Hey! You mentioned me. Here's what I can do...

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai explain the PR above and diff the changes

@agentwaspai

agentwaspai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Hey! You mentioned me. Here's what I can do...

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai

@agentwaspai

agentwaspai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Hey! You mentioned me. Here's what I can do...

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai

@agentwaspai

agentwaspai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Hey! You mentioned me. Here's what I can do...

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai

@agentwaspai

agentwaspai Bot commented Jul 1, 2026

Copy link
Copy Markdown

Hey! You mentioned me. Here's what I can do...

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /review

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Raw PR Data

  • Files changed: 18
  • Commits: 13
  • Comments: 30
  • Review comments: 0
  • Reviews: 2

Files

  • app.yml [modified] +13 -6
  • deploy.yml [renamed] +0 -0
  • index.js [modified] +90 -49
  • intro.js [added] +15 -0
  • k8s/cognee-deployment.yml [added] +115 -0
  • memory/__pycache__/cognee.cpython-311.pyc [added] +0 -0
  • memory/api/__pycache__/main.cpython-311.pyc [added] +0 -0
  • memory/api/main.py [added] +58 -0
  • memory/cloud.test.py [added] +0 -0
  • memory/config/__pycache__/rules.cpython-311.pyc [added] +0 -0
  • memory/config/rules.py [added] +11 -0
  • memory/llm.py [added] +0 -0
  • memory/main.py [added] +146 -0
  • memory/rules/ask.txt [added] +18 -0
  • memory/rules/claude.md [added] +73 -0
  • memory/rules/review.txt [added] +38 -0
  • memory/rules/rules.txt [added] +53 -0
  • test/index.test.js [modified] +65 -64

Commits

  • 8e72dfc Merge pull request memory #2 from inline-arc/testing

memory (Atharv More)

  • 42afa12 addup (inline-arc)
  • 215ebb2 events (inline-arc)
  • ae5c289 first call (inline-arc)
  • 079fdf7 Merge branch 'master' of https://github.com/inline-arc/Context-MemoryPR (inline-arc)
  • 4d50020 venv (inline-arc)
  • b1244aa rules.txt (inline-arc)
  • da21db3 new rules.txt (inline-arc)
  • 14fcecd more rules (inline-arc)
  • fd98e80 yaml (inline-arc)
  • b845cf5 cloud (inline-arc)
  • ff7cdbb call (inline-arc)
  • 43ad77a metadata (inline-arc)

Diff preview

diff --git a/app.yml b/app.yml
index e5418ba..d1974ae 100644
--- a/app.yml
+++ b/app.yml
@@ -19,11 +19,11 @@ default_events:
   # - create
   # - delete
   - deployment
-  # - deployment_status
+  - deployment_status
   # - fork
   # - gollum
-  # - issue_comment
-  # - issues
+  - issue_comment
+  - issues
   # - label
   # - milestone
   # - member
@@ -36,8 +36,10 @@ default_events:
   # - project_column
   # - public
   - pull_request
-# - pull_request_review
-# - pull_request_review_comment
+  - pull_request_review
+  - pull_request_review_comment
+  - discussion          # added this
+  - discussion_comment  # added this
 # - push
 # - release
 # - repository
@@ -70,7 +72,7 @@ default_permissions:
 
   # Issues and related comments, assignees, labels, and milestones.
   # https://developer.github.com/v3/apps/permissions/#permission-on-issues
-  # issues: write
+  issues: write
 
   # Search repositories, list collaborators, and access repository '.
   # https://developer.github.co

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Error: Ingest failed: 500

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /review

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Raw PR Data

  • Files changed: 18
  • Commits: 13
  • Comments: 30
  • Review comments: 0
  • Reviews: 2

Files

  • app.yml [modified] +13 -6
  • deploy.yml [renamed] +0 -0
  • index.js [modified] +90 -49
  • intro.js [added] +15 -0
  • k8s/cognee-deployment.yml [added] +115 -0
  • memory/__pycache__/cognee.cpython-311.pyc [added] +0 -0
  • memory/api/__pycache__/main.cpython-311.pyc [added] +0 -0
  • memory/api/main.py [added] +58 -0
  • memory/cloud.test.py [added] +0 -0
  • memory/config/__pycache__/rules.cpython-311.pyc [added] +0 -0
  • memory/config/rules.py [added] +11 -0
  • memory/llm.py [added] +0 -0
  • memory/main.py [added] +146 -0
  • memory/rules/ask.txt [added] +18 -0
  • memory/rules/claude.md [added] +73 -0
  • memory/rules/review.txt [added] +38 -0
  • memory/rules/rules.txt [added] +53 -0
  • test/index.test.js [modified] +65 -64

Commits

  • 8e72dfc Merge pull request memory #2 from inline-arc/testing

memory (Atharv More)

  • 42afa12 addup (inline-arc)
  • 215ebb2 events (inline-arc)
  • ae5c289 first call (inline-arc)
  • 079fdf7 Merge branch 'master' of https://github.com/inline-arc/Context-MemoryPR (inline-arc)
  • 4d50020 venv (inline-arc)
  • b1244aa rules.txt (inline-arc)
  • da21db3 new rules.txt (inline-arc)
  • 14fcecd more rules (inline-arc)
  • fd98e80 yaml (inline-arc)
  • b845cf5 cloud (inline-arc)
  • ff7cdbb call (inline-arc)
  • 43ad77a metadata (inline-arc)

Diff preview

diff --git a/app.yml b/app.yml
index e5418ba..d1974ae 100644
--- a/app.yml
+++ b/app.yml
@@ -19,11 +19,11 @@ default_events:
   # - create
   # - delete
   - deployment
-  # - deployment_status
+  - deployment_status
   # - fork
   # - gollum
-  # - issue_comment
-  # - issues
+  - issue_comment
+  - issues
   # - label
   # - milestone
   # - member
@@ -36,8 +36,10 @@ default_events:
   # - project_column
   # - public
   - pull_request
-# - pull_request_review
-# - pull_request_review_comment
+  - pull_request_review
+  - pull_request_review_comment
+  - discussion          # added this
+  - discussion_comment  # added this
 # - push
 # - release
 # - repository
@@ -70,7 +72,7 @@ default_permissions:
 
   # Issues and related comments, assignees, labels, and milestones.
   # https://developer.github.com/v3/apps/permissions/#permission-on-issues
-  # issues: write
+  issues: write
 
   # Search repositories, list collaborators, and access repository '.
   # https://developer.github.co

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

PR #3 data stored in memory and ready for review.

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /review

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Raw PR Data

  • Files changed: 18
  • Commits: 13
  • Comments: 30
  • Review comments: 0
  • Reviews: 2

Files

  • app.yml [modified] +13 -6
  • deploy.yml [renamed] +0 -0
  • index.js [modified] +90 -49
  • intro.js [added] +15 -0
  • k8s/cognee-deployment.yml [added] +115 -0
  • memory/__pycache__/cognee.cpython-311.pyc [added] +0 -0
  • memory/api/__pycache__/main.cpython-311.pyc [added] +0 -0
  • memory/api/main.py [added] +58 -0
  • memory/cloud.test.py [added] +0 -0
  • memory/config/__pycache__/rules.cpython-311.pyc [added] +0 -0
  • memory/config/rules.py [added] +11 -0
  • memory/llm.py [added] +0 -0
  • memory/main.py [added] +146 -0
  • memory/rules/ask.txt [added] +18 -0
  • memory/rules/claude.md [added] +73 -0
  • memory/rules/review.txt [added] +38 -0
  • memory/rules/rules.txt [added] +53 -0
  • test/index.test.js [modified] +65 -64

Commits

  • 8e72dfc Merge pull request memory #2 from inline-arc/testing

memory (Atharv More)

  • 42afa12 addup (inline-arc)
  • 215ebb2 events (inline-arc)
  • ae5c289 first call (inline-arc)
  • 079fdf7 Merge branch 'master' of https://github.com/inline-arc/Context-MemoryPR (inline-arc)
  • 4d50020 venv (inline-arc)
  • b1244aa rules.txt (inline-arc)
  • da21db3 new rules.txt (inline-arc)
  • 14fcecd more rules (inline-arc)
  • fd98e80 yaml (inline-arc)
  • b845cf5 cloud (inline-arc)
  • ff7cdbb call (inline-arc)
  • 43ad77a metadata (inline-arc)

Diff preview

diff --git a/app.yml b/app.yml
index e5418ba..d1974ae 100644
--- a/app.yml
+++ b/app.yml
@@ -19,11 +19,11 @@ default_events:
   # - create
   # - delete
   - deployment
-  # - deployment_status
+  - deployment_status
   # - fork
   # - gollum
-  # - issue_comment
-  # - issues
+  - issue_comment
+  - issues
   # - label
   # - milestone
   # - member
@@ -36,8 +36,10 @@ default_events:
   # - project_column
   # - public
   - pull_request
-# - pull_request_review
-# - pull_request_review_comment
+  - pull_request_review
+  - pull_request_review_comment
+  - discussion          # added this
+  - discussion_comment  # added this
 # - push
 # - release
 # - repository
@@ -70,7 +72,7 @@ default_permissions:
 
   # Issues and related comments, assignees, labels, and milestones.
   # https://developer.github.com/v3/apps/permissions/#permission-on-issues
-  # issues: write
+  issues: write
 
   # Search repositories, list collaborators, and access repository '.
   # https://developer.github.co

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

PR #3 data stored in memory and ready for review.

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is pr context ?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Error: Recall failed: 500

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is pr context ?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Error: Recall failed: 500

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is pr context ?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

AgentWasp AI

[1] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': '## PR Context\n\n- Definition – The PR (pull‑request) context is the surrounding information that explains why a PR exists and what it intends to change. \n- Typical elements \n - A brief description of the feature, bug‑fix, or refactor. \n - The list of files or modules affected. \n - References to related tickets, issues, or design documents. \n - Any assumptions, constraints, or migration steps required. \n- Why it matters – Reviewers use the context to evaluate whether the changes align with the intended goal, to spot missing tests or documentation, and to assess impact on the rest of the codebase. ', 'score': None, 'dataset_id': 'dc73590d-b20f-548b-9bcf-edd659f0a743', 'dataset_name': 'default_dataset', 'metadata': {}, 'raw': {'value': '## PR Context\n\n- Definition – The PR (pull‑request) context is the surrounding information that explains why a PR exists and what it intends to change. \n- Typical elements \n - A brief description of the feature, bug‑fix, or refactor. \n - The list of files or modules affected. \n - References to related tickets, issues, or design documents. \n - Any assumptions, constraints, or migration steps required. \n- Why it matters – Reviewers use the context to evaluate whether the changes align with the intended goal, to spot missing tests or documentation, and to assess impact on the rest of the codebase. '}, 'structured': None, 'source': 'graph'}


[2] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': '## PR Context\n\n- Definition – The set of information that describes a pull request and provides the background needed to evaluate it. \n- Typical components \n - PR metadata: number, title, description, author, target branch. \n - List of changed files and the line‑by‑line diff. \n - Commit history (messages, SHAs) for the PR. \n - Associated labels, linked tickets/issues, and any review comments. \n\nThis context lets reviewers understand why the PR exists, what it changes, and how it fits into the larger codebase.', 'score': None, 'dataset_id': '853aa850-f8a8-5fe3-8fcd-90e7cac6a9cf', 'dataset_name': 'agent_memory_dataset', 'metadata': {}, 'raw': {'value': '## PR Context\n\n- Definition – The set of information that describes a pull request and provides the background needed to evaluate it. \n- Typical components \n - PR metadata: number, title, description, author, target branch. \n - List of changed files and the line‑by‑line diff. \n - Commit history (messages, SHAs) for the PR. \n - Associated labels, linked tickets/issues, and any review comments. \n\nThis context lets reviewers understand why the PR exists, what it changes, and how it fits into the larger codebase.'}, 'structured': None, 'source': 'graph'}


[3] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': '## PR Context\n\n- Definition – The set of information that describes why a pull request (PR) was opened and what it intends to change. \n- Typical elements \n - PR metadata: number, title, description, author, target branch. \n - List of files or modules affected. \n - The diff – line‑by‑line additions and deletions. \n - Commit history – ordered commit messages and SHAs for the PR. \n - Associated data: related tickets/issues, labels, reviewers, and review comments. \n- Why it matters – Provides reviewers with the background needed to evaluate whether the changes align with the intended goal, to spot missing tests or documentation, and to gauge the impact on the rest of the codebase.\n\nSummary: PR context is the collection of metadata, diff, commits, and related references that together explain a PR’s purpose and scope, enabling informed code‑review decisions.', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': '## PR Context\n\n- Definition – The set of information that describes why a pull request (PR) was opened and what it intends to change. \n- Typical elements \n - PR metadata: number, title, description, author, target branch. \n - List of files or modules affected. \n - The diff – line‑by‑line additions and deletions. \n - Commit history – ordered commit messages and SHAs for the PR. \n - Associated data: related tickets/issues, labels, reviewers, and review comments. \n- Why it matters – Provides reviewers with the background needed to evaluate whether the changes align with the intended goal, to spot missing tests or documentation, and to gauge the impact on the rest of the codebase.\n\nSummary: PR context is the collection of metadata, diff, commits, and related references that together explain a PR’s purpose and scope, enabling informed code‑review decisions.'}, 'structured': None, 'source': 'graph'}


[4] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': 'PR context is the set of information that describes a pull request and is used to ground answers or code‑review decisions. It typically includes:\n\n- Metadata – PR number, title, description, author, target branch. \n- Changed files – list of files touched, additions/deletions. \n- Diff – the actual line‑by‑line changes (the ---/+++ view). \n- Commit history – ordered commit messages and SHA’s for the PR. \n- Associated metadata – labels, reviewers, review comments, and any related events.\n\nIn the knowledge graph, pr #5 (e.g., “Fix adding error handling to getUserById”) together with its diff and author is an example of PR context that the Ask and Review agents use to ground their responses.', 'score': None, 'dataset_id': 'a0fe2a1f-ca9b-5d79-b353-0303d86ec04c', 'dataset_name': 'default_session', 'metadata': {}, 'raw': {'value': 'PR context is the set of information that describes a pull request and is used to ground answers or code‑review decisions. It typically includes:\n\n- Metadata – PR number, title, description, author, target branch. \n- Changed files – list of files touched, additions/deletions. \n- Diff – the actual line‑by‑line changes (the ---/+++ view). \n- Commit history – ordered commit messages and SHA’s for the PR. \n- Associated metadata – labels, reviewers, review comments, and any related events.\n\nIn the knowledge graph, pr #5 (e.g., “Fix adding error handling to getUserById”) together with its diff and author is an example of PR context that the Ask and Review agents use to ground their responses.'}, 'structured': None, 'source': 'graph'}

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is the PR commit history?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Error: Recall failed: 500

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is the PR commit history?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

AgentWasp AI

[1] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': '## PR Commit History\n\n- Definition – The PR (pull‑request) commit history is the ordered list of commits that make up a pull request. Each entry records a snapshot of the code (its SHA), the author, timestamp, and commit message, and links to its parent commit(s).\n\n- Purpose – Shows how the changes evolved, lets reviewers trace the introduction of each feature, bug‑fix, or refactor, and helps identify where regressions or unwanted modifications may have been added.\n\n- Typical elements (from the provided graph) \n - Commits: be5cd5b3, 057b9821, 2c4d5449, ae5c2895…, 42afa121…, 215ebb24…, 98ec5e00, 8e72dfc, 32ccbac7, 6c1135c0, etc. \n - Parent relationships: \n - ae5c2895… → parent 42afa121… \n - 42afa121… → parent 9096906d… \n\n- Best‑practice considerations \n - Write clear, descriptive commit messages (what changed and why). \n - Keep commits atomic: each commit should represent a single logical change. \n - Ensure the history is logical and builds successfully after each commit; squash or rebase noisy histories when the project prefers a linear timeline. \n\nSummary: The PR commit history is the sequential record of all commits (with their metadata and parent links) that constitute a pull request, providing essential context for understanding and reviewing the changes.', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': '## PR Commit History\n\n- Definition – The PR (pull‑request) commit history is the ordered list of commits that make up a pull request. Each entry records a snapshot of the code (its SHA), the author, timestamp, and commit message, and links to its parent commit(s).\n\n- Purpose – Shows how the changes evolved, lets reviewers trace the introduction of each feature, bug‑fix, or refactor, and helps identify where regressions or unwanted modifications may have been added.\n\n- Typical elements (from the provided graph) \n - Commits: be5cd5b3, 057b9821, 2c4d5449, ae5c2895…, 42afa121…, 215ebb24…, 98ec5e00, 8e72dfc, 32ccbac7, 6c1135c0, etc. \n - Parent relationships: \n - ae5c2895… → parent 42afa121… \n - 42afa121… → parent 9096906d… \n\n- Best‑practice considerations \n - Write clear, descriptive commit messages (what changed and why). \n - Keep commits atomic: each commit should represent a single logical change. \n - Ensure the history is logical and builds successfully after each commit; squash or rebase noisy histories when the project prefers a linear timeline. \n\nSummary: The PR commit history is the sequential record of all commits (with their metadata and parent links) that constitute a pull request, providing essential context for understanding and reviewing the changes.'}, 'structured': None, 'source': 'graph'}

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what changes are made in this pr #3

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

AgentWasp AI

[1] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': 'PR\u202f#3 – Summary of changes\n\n- Configuration (app.yml)\n - Enabled deployment_status, issue_comment, issues.\n - Added pull_request_review, pull_request_review_comment, discussion, and discussion_comment events.\n - Set issues permission to write and added discussions: write permission.\n\n- Probot app (index.js)\n - Replaced the demo deployment script with a full‑featured bot:\n - Defines BOT_NAME = "@agentwaspai".\n - Listens for issue_comment.created containing “/review”.\n - Pulls PR diff, file list, comments, review comments, reviews, and commits in parallel.\n - Logs raw data and posts a summarized comment back to the PR.\n - Added error handling for non‑PR comments.\n\n- New files\n - intro.js – introductory message and command table for the bot.\n - k8s/cognee-deployment.yml – Kubernetes manifest for the Cognee memory service.\n - Numerous memory package files:\n - memory/api/main.py\n - memory/cloud.test.py\n - memory/config/rules.py\n - memory/llm.py\n - memory/main.py\n - Rule files (ask.txt, claude.md, review.txt, rules.txt)\n - Corresponding __pycache__ binaries (auto‑generated).\n - Updated test suite file test/index.test.js.\n\n- Renames\n - memory/cognee.py renamed to deploy.yml (pure rename, no content change).\n\n- Other modifications\n - Minor tweaks to app.yml permissions block (added discussion permissions).\n - Updated import/export style and logging messages in the bot code.\n\nOverall impact: The PR expands the repository with a functional review bot, adds extensive memory‑related modules and tests, introduces Kubernetes deployment configuration, and refines the GitHub App permissions and event handling.', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': 'PR\u202f#3 – Summary of changes\n\n- Configuration (app.yml)\n - Enabled deployment_status, issue_comment, issues.\n - Added pull_request_review, pull_request_review_comment, discussion, and discussion_comment events.\n - Set issues permission to write and added discussions: write permission.\n\n- Probot app (index.js)\n - Replaced the demo deployment script with a full‑featured bot:\n - Defines BOT_NAME = "@agentwaspai".\n - Listens for issue_comment.created containing “/review”.\n - Pulls PR diff, file list, comments, review comments, reviews, and commits in parallel.\n - Logs raw data and posts a summarized comment back to the PR.\n - Added error handling for non‑PR comments.\n\n- New files\n - intro.js – introductory message and command table for the bot.\n - k8s/cognee-deployment.yml – Kubernetes manifest for the Cognee memory service.\n - Numerous memory package files:\n - memory/api/main.py\n - memory/cloud.test.py\n - memory/config/rules.py\n - memory/llm.py\n - memory/main.py\n - Rule files (ask.txt, claude.md, review.txt, rules.txt)\n - Corresponding __pycache__ binaries (auto‑generated).\n - Updated test suite file test/index.test.js.\n\n- Renames\n - memory/cognee.py renamed to deploy.yml (pure rename, no content change).\n\n- Other modifications\n - Minor tweaks to app.yml permissions block (added discussion permissions).\n - Updated import/export style and logging messages in the bot code.\n\nOverall impact: The PR expands the repository with a functional review bot, adds extensive memory‑related modules and tests, introduces Kubernetes deployment configuration, and refines the GitHub App permissions and event handling.'}, 'structured': None, 'source': 'graph'}

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is the PR commit history?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

AgentWasp AI

[1] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': 'PR commit history – the ordered list of commits that make up the pull‑request, showing each snapshot (SHA), its author/date (when known) and how the commits are linked through parent relationships.\n\n### Commits present in the graph \n\n| SHA (short) | Message / note | Parent (if known) |\n|-------------|----------------|-------------------|\n| 9096906d… | (base commit – no details) | – |\n| 42afa121… | – | 9096906d… |\n| ae5c2895… | first call (2026‑06‑26) | 42afa121… |\n| 215ebb24… | events (2026‑06‑26) | – |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | – (author unknown, 2026‑07‑06) | – |\n| 057b9821 | – (author unknown) | – |\n| 2c4d5449 | – (author unknown) | – |\n| 9f5b6db1 | – (author unknown) | – |\n| 98ec5e00 | – (author unknown) | – |\n| 32ccbac7 | – (author unknown) | – |\n| 6c1135c0 | – (author unknown) | – |\n| 8e72dfc5137fa… | – (repo‑specific entry) | – |\n\n### Key points for reviewers\n- Lineage: The only explicit chain is 9096906d… → 42afa121… → ae5c2895…. \n- Atomicity: Each commit should represent a single logical change (e.g., “first call”, “events”, merge). \n- Clarity: Commit messages such as first call and events give context; others need clearer messages before merging. \n- Order: Verify that the PR builds after each commit in the above sequence; consider squashing or rebasing noisy, unrelated commits into a cleaner linear history.', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': 'PR commit history – the ordered list of commits that make up the pull‑request, showing each snapshot (SHA), its author/date (when known) and how the commits are linked through parent relationships.\n\n### Commits present in the graph \n\n| SHA (short) | Message / note | Parent (if known) |\n|-------------|----------------|-------------------|\n| 9096906d… | (base commit – no details) | – |\n| 42afa121… | – | 9096906d… |\n| ae5c2895… | first call (2026‑06‑26) | 42afa121… |\n| 215ebb24… | events (2026‑06‑26) | – |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | – (author unknown, 2026‑07‑06) | – |\n| 057b9821 | – (author unknown) | – |\n| 2c4d5449 | – (author unknown) | – |\n| 9f5b6db1 | – (author unknown) | – |\n| 98ec5e00 | – (author unknown) | – |\n| 32ccbac7 | – (author unknown) | – |\n| 6c1135c0 | – (author unknown) | – |\n| 8e72dfc5137fa… | – (repo‑specific entry) | – |\n\n### Key points for reviewers\n- Lineage: The only explicit chain is 9096906d… → 42afa121… → ae5c2895…. \n- Atomicity: Each commit should represent a single logical change (e.g., “first call”, “events”, merge). \n- Clarity: Commit messages such as first call and events give context; others need clearer messages before merging. \n- Order: Verify that the PR builds after each commit in the above sequence; consider squashing or rebasing noisy, unrelated commits into a cleaner linear history.'}, 'structured': None, 'source': 'graph'}

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is the PR commit history?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

AgentWasp AI

[1] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': 'PR commit history – the ordered series of commits that constitute a pull‑request, each showing a snapshot of the code (SHA), author/date, and a commit message. It lets reviewers see how the change evolved and trace where any issue was introduced.\n\nCommit sequence extracted from the graph (earliest → latest):\n\n| SHA (short) | Message / note | Parent |\n|-------------|----------------|--------|\n| 9096906d… | (base commit – no details) | – |\n| 42afa121… | – | 9096906d… |\n| ae5c2895… | “first call” (2026‑06‑26) | 42afa121… |\n| 215ebb24… | “events” (2026‑06‑26) | – |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | – (author unknown, 2026‑07‑06) | – |\n| 057b9821 | – (author unknown) | – |\n| 2c4d5449 | – (author unknown) | – |\n| 9f5b6db1 | – (author unknown) | – |\n| 98ec5e00 | – (author unknown) | – |\n| 32ccbac7 | – (author unknown) | – |\n| 6c1135c0 | – (author unknown) | – |\n| 8e72dfc5137fa… | – (repo‑specific entry) | – |\n\nKey points for reviewers\n\n* The only explicit lineage in the graph is 9096906d… → 42afa121… → ae5c2895…; the rest appear as independent commits within the PR.\n* Commits with clear messages (“first call”, “events”, merge) provide useful context; the others should have descriptive messages added before merging.\n* Ensure the PR builds and passes tests after each commit, or consider squashing/rebasing to produce a cleaner linear history.', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': 'PR commit history – the ordered series of commits that constitute a pull‑request, each showing a snapshot of the code (SHA), author/date, and a commit message. It lets reviewers see how the change evolved and trace where any issue was introduced.\n\nCommit sequence extracted from the graph (earliest → latest):\n\n| SHA (short) | Message / note | Parent |\n|-------------|----------------|--------|\n| 9096906d… | (base commit – no details) | – |\n| 42afa121… | – | 9096906d… |\n| ae5c2895… | “first call” (2026‑06‑26) | 42afa121… |\n| 215ebb24… | “events” (2026‑06‑26) | – |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | – (author unknown, 2026‑07‑06) | – |\n| 057b9821 | – (author unknown) | – |\n| 2c4d5449 | – (author unknown) | – |\n| 9f5b6db1 | – (author unknown) | – |\n| 98ec5e00 | – (author unknown) | – |\n| 32ccbac7 | – (author unknown) | – |\n| 6c1135c0 | – (author unknown) | – |\n| 8e72dfc5137fa… | – (repo‑specific entry) | – |\n\nKey points for reviewers\n\n* The only explicit lineage in the graph is 9096906d… → 42afa121… → ae5c2895…; the rest appear as independent commits within the PR.\n* Commits with clear messages (“first call”, “events”, merge) provide useful context; the others should have descriptive messages added before merging.\n* Ensure the PR builds and passes tests after each commit, or consider squashing/rebasing to produce a cleaner linear history.'}, 'structured': None, 'source': 'graph'}

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is the PR commit history?

@agentwaspai

agentwaspai Bot commented Jul 6, 2026

Copy link
Copy Markdown

AgentWasp AI

[1] {'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': 'PR commit history – the ordered series of commits that make up the pull‑request, each showing a snapshot of the code (SHA), author/date (when known) and a commit message.\n\nCommits present in this PR (earliest → latest, based on the parent links that are known):\n\n| SHA (short) | Message / note | Author / date |\n|-------------|----------------|---------------|\n| 9096906d… | (base commit – no details) | – |\n| 42afa121… | – | – |\n| ae5c2895… | “first call” (2026‑06‑26) | unknown |\n| 215ebb24… | “events” (2026‑06‑26) | unknown |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | – | unknown (2026‑07‑06) |\n| 057b9821 | – | unknown |\n| 2c4d5449 | – | unknown |\n| 9f5b6db1 | – | unknown |\n| 98ec5e00 | – | unknown |\n| 32ccbac7 | – | unknown |\n| 6c1135c0 | – | unknown |\n| 8e72dfc5137fa… | – | – |\n\nKey points for reviewers\n\n* The only explicit lineage is 9096906d… → 42afa121… → ae5c2895…; the remaining commits appear as independent entries within the PR. \n* Commit messages that are clear (“first call”, “events”, merge) give useful context; the other commits should have descriptive messages added before merging. \n* Verify that the code builds and passes tests after each commit, or consider squashing/rebasing to produce a cleaner linear history. ', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': 'PR commit history – the ordered series of commits that make up the pull‑request, each showing a snapshot of the code (SHA), author/date (when known) and a commit message.\n\nCommits present in this PR (earliest → latest, based on the parent links that are known):\n\n| SHA (short) | Message / note | Author / date |\n|-------------|----------------|---------------|\n| 9096906d… | (base commit – no details) | – |\n| 42afa121… | – | – |\n| ae5c2895… | “first call” (2026‑06‑26) | unknown |\n| 215ebb24… | “events” (2026‑06‑26) | unknown |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | – | unknown (2026‑07‑06) |\n| 057b9821 | – | unknown |\n| 2c4d5449 | – | unknown |\n| 9f5b6db1 | – | unknown |\n| 98ec5e00 | – | unknown |\n| 32ccbac7 | – | unknown |\n| 6c1135c0 | – | unknown |\n| 8e72dfc5137fa… | – | – |\n\nKey points for reviewers\n\n* The only explicit lineage is 9096906d… → 42afa121… → ae5c2895…; the remaining commits appear as independent entries within the PR. \n* Commit messages that are clear (“first call”, “events”, merge) give useful context; the other commits should have descriptive messages added before merging. \n* Verify that the code builds and passes tests after each commit, or consider squashing/rebasing to produce a cleaner linear history. '}, 'structured': None, 'source': 'graph'}

@inline-arc

Copy link
Copy Markdown
Owner Author

@agentwaspai /ask what is the PR commit history?

@agentwaspai

agentwaspai Bot commented Jul 9, 2026

Copy link
Copy Markdown

AgentWasp Recall



Question: what is the PR commit history?



{'kind': 'graph_completion', 'search_type': 'GRAPH_COMPLETION', 'text': 'PR commit history – the ordered series of commits that make up a pull‑request. Each entry records a snapshot of the code (SHA), the author/date (when known) and a commit message, showing how the change evolved.\n\n### Commits present in this PR (earliest → latest)\n\n| SHA (short) | Message / note | Author / date |\n|-------------|----------------|---------------|\n| 9096906d… | (no details) | – |\n| 42afa121… | (no message) | – |\n| ae5c2895… | “first call” | 2026‑06‑26 |\n| 215ebb24… | “events” | 2026‑06‑26 |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | (no message) | unknown, 2026‑07‑06 |\n| 057b9821 | (no message) | unknown |\n| 2c4d5449 | (no message) | unknown |\n| 9f5b6db1 | (no message) | unknown |\n| 98ec5e00 | (no message) | unknown |\n| 32ccbac7 | (no message) | unknown |\n| 6c1135c0 | (no message) | unknown |\n| 8e72dfc5137fa… | (no message) | – |\n\n### What to look for when reviewing\n\n- Clear commit messages – “first call” and “events” are useful; the other commits should get descriptive messages that explain what changed and why.\n- Logical grouping – each commit should represent a single, coherent change (e.g., a feature, a bug‑fix, a refactor). If many tiny or unrelated commits exist, consider squashing or rebasing to a cleaner linear history.\n- Buildability – the code should compile/passing tests after each commit; otherwise the history may need to be reordered or cleaned up. \n\nA well‑structured commit history gives reviewers a clear trail to trace the introduction of each change, spot regressions, and decide whether the PR is ready to merge.', 'score': None, 'dataset_id': '0ccc96d3-9345-54b5-b990-b67769ce87f8', 'dataset_name': 'repo-inline-arc-AgentwaspAi', 'metadata': {}, 'raw': {'value': 'PR commit history – the ordered series of commits that make up a pull‑request. Each entry records a snapshot of the code (SHA), the author/date (when known) and a commit message, showing how the change evolved.\n\n### Commits present in this PR (earliest → latest)\n\n| SHA (short) | Message / note | Author / date |\n|-------------|----------------|---------------|\n| 9096906d… | (no details) | – |\n| 42afa121… | (no message) | – |\n| ae5c2895… | “first call” | 2026‑06‑26 |\n| 215ebb24… | “events” | 2026‑06‑26 |\n| 8e72dfc | Merge pull request #2 from inline‑arc/testing | – |\n| be5cd5b3 | (no message) | unknown, 2026‑07‑06 |\n| 057b9821 | (no message) | unknown |\n| 2c4d5449 | (no message) | unknown |\n| 9f5b6db1 | (no message) | unknown |\n| 98ec5e00 | (no message) | unknown |\n| 32ccbac7 | (no message) | unknown |\n| 6c1135c0 | (no message) | unknown |\n| 8e72dfc5137fa… | (no message) | – |\n\n### What to look for when reviewing\n\n- Clear commit messages – “first call” and “events” are useful; the other commits should get descriptive messages that explain what changed and why.\n- Logical grouping – each commit should represent a single, coherent change (e.g., a feature, a bug‑fix, a refactor). If many tiny or unrelated commits exist, consider squashing or rebasing to a cleaner linear history.\n- Buildability – the code should compile/passing tests after each commit; otherwise the history may need to be reordered or cleaned up. \n\nA well‑structured commit history gives reviewers a clear trail to trace the introduction of each change, spot regressions, and decide whether the PR is ready to merge.'}, 'structured': None, 'source': 'graph'}

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