Skip to content

Completing task#10

Merged
portiajefferson merged 6 commits intomainfrom
copilot/sync-main-and-refresh-ingestion
Mar 28, 2026
Merged

Completing task#10
portiajefferson merged 6 commits intomainfrom
copilot/sync-main-and-refresh-ingestion

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 28, 2026

Pull request created by AI Agent

Copilot AI and others added 5 commits March 28, 2026 05:02
…in backend context

Co-authored-by: portiajefferson <67895258+portiajefferson@users.noreply.github.com>
…dentials

Co-authored-by: portiajefferson <67895258+portiajefferson@users.noreply.github.com>
Co-authored-by: portiajefferson <67895258+portiajefferson@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

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

This PR extends the backend’s Azure-powered RAG capabilities by introducing document management support, adding ingestion tooling, and updating Azure client configuration and dependencies.

Changes:

  • Adds /documents routing to the backend Express server and fixes the server port variable usage.
  • Updates backend dependencies to include Azure OpenAI, Azure Search, Form Recognizer, plus upload utilities (multer, uuid).
  • Adds an ingest script at the repo root to run the backend ingestion command.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Adds a root-level ingest script that invokes the backend ingestion script.
backend/server.js Registers documents routes and fixes PORT usage in app.listen.
backend/package.json Adds Azure/OpenAI/Search/Form Recognizer + upload-related dependencies.
backend/package-lock.json Locks newly added backend dependencies.
backend/config/azureConfig.js Changes Azure client initialization to conditional (may return null) and exports additional clients.
Files not reviewed (1)
  • backend/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)

backend/server.js:74

  • The server banner advertises POST /query and GET /audit, but the mounted routers expose different subpaths (e.g., backend/routes/query.js defines POST /rag, and backend/routes/audit.js defines GET /audit-log and /audit-log/schema). With the current app.use('/query', queryRoutes) and app.use('/audit', auditRoutes) mounts, the actual endpoints become /query/rag and /audit/audit-log, etc. Please either adjust the mounts or update the routers/logged endpoint list so they match.
app.listen(PORT, () => {
  console.log(`\n🚀 Grounded Knowledge Assistant API running on port ${PORT}`);
  console.log(`\n📋 Available endpoints:`);
  console.log(`   POST   /query          - Ask a question`);
  console.log(`   GET    /audit          - View audit logs`);
  console.log(`   GET    /audit/stats    - View audit statistics`);
  console.log(`   GET    /audit/export   - Export audit logs`);

@portiajefferson
Copy link
Copy Markdown
Contributor

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

@portiajefferson The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: f7543c00-47a8-4804-8a9b-be38c4262c9e

2 similar comments
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

@portiajefferson The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: f7543c00-47a8-4804-8a9b-be38c4262c9e

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

@portiajefferson The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: f7543c00-47a8-4804-8a9b-be38c4262c9e

@portiajefferson portiajefferson marked this pull request as ready for review March 28, 2026 05:49
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

@portiajefferson The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: f7543c00-47a8-4804-8a9b-be38c4262c9e

2 similar comments
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

@portiajefferson The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: f7543c00-47a8-4804-8a9b-be38c4262c9e

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 28, 2026

@portiajefferson The model claude-sonnet-4.6 is not available for your account. This can happen if the model was disabled by your organization's policy or if your Copilot plan doesn't include access to it.

You can try again without specifying a model (just @copilot) to use the default, or choose a different model from the model picker.

If you want to contact GitHub about this error, please mention the following identifier so they can better serve you: f7543c00-47a8-4804-8a9b-be38c4262c9e

Agent-Logs-Url: https://github.com/TriResolve-AI/grounded-knowledge-assistant/sessions/7d65f388-bdd0-4dca-92f9-eec31fb5c06b

Co-authored-by: portiajefferson <67895258+portiajefferson@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

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

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

Files not reviewed (1)
  • backend/package-lock.json: Language not supported
Comments suppressed due to low confidence (1)

backend/server.js:74

  • The startup console output lists endpoints that don't match the actual mounted routes. For example, queryRoutes defines POST /rag (so the full path is /query/rag), and auditRoutes defines /audit-log + /audit-log/schema (so paths are /audit/audit-log and /audit/audit-log/schema). Updating these log lines (and ideally the / root endpoint listing) will prevent confusion for API consumers and demo scripts.
  console.log(`\n📋 Available endpoints:`);
  console.log(`   POST   /query          - Ask a question`);
  console.log(`   GET    /audit          - View audit logs`);
  console.log(`   GET    /audit/stats    - View audit statistics`);
  console.log(`   GET    /audit/export   - Export audit logs`);

@portiajefferson portiajefferson merged commit 89f91ba into main Mar 28, 2026
1 check failed
@portiajefferson portiajefferson deleted the copilot/sync-main-and-refresh-ingestion branch March 28, 2026 06:19
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.

3 participants