feat(ai-scribe): Fix some comments of coderabbit
This commit is contained in:
@@ -27,6 +27,13 @@ mixin AiScribeMixin {
|
|||||||
|
|
||||||
if (scribeEndpoint == null || scribeEndpoint.isEmpty) return;
|
if (scribeEndpoint == null || scribeEndpoint.isEmpty) return;
|
||||||
|
|
||||||
|
// Validate endpoint format
|
||||||
|
if (Uri.tryParse(scribeEndpoint)?.hasAbsolutePath != true) {
|
||||||
|
logError(
|
||||||
|
'AiScribeMixin::injectAIScribeBindings(): Invalid endpoint format: $scribeEndpoint');
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
AIScribeBindings(scribeEndpoint).dependencies();
|
AIScribeBindings(scribeEndpoint).dependencies();
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
logError('AiScribeMixin::injectAIScribeBindings(): $e');
|
logError('AiScribeMixin::injectAIScribeBindings(): $e');
|
||||||
|
|||||||
@@ -35,7 +35,7 @@ extension SessionExtensions on Session {
|
|||||||
linagoraDownloadAllCapability: DownloadAllCapability.deserialize,
|
linagoraDownloadAllCapability: DownloadAllCapability.deserialize,
|
||||||
capabilityServerSettings: SettingsCapability.deserialize,
|
capabilityServerSettings: SettingsCapability.deserialize,
|
||||||
linagoraSaaSCapability: SaaSAccountCapability.deserialize,
|
linagoraSaaSCapability: SaaSAccountCapability.deserialize,
|
||||||
AiScribeConstants.aiCapability: AICapability.deserialize,
|
AiScribeConstants.aiCapability: AICapability.fromJson,
|
||||||
};
|
};
|
||||||
|
|
||||||
Map<String, dynamic> toJson() {
|
Map<String, dynamic> toJson() {
|
||||||
|
|||||||
@@ -1,6 +1,5 @@
|
|||||||
import 'package:core/presentation/state/failure.dart';
|
import 'package:core/presentation/state/failure.dart';
|
||||||
import 'package:core/presentation/state/success.dart';
|
import 'package:core/presentation/state/success.dart';
|
||||||
import 'package:core/utils/app_logger.dart';
|
|
||||||
import 'package:dartz/dartz.dart';
|
import 'package:dartz/dartz.dart';
|
||||||
import 'package:scribe/scribe/ai/domain/constants/ai_prompts.dart';
|
import 'package:scribe/scribe/ai/domain/constants/ai_prompts.dart';
|
||||||
import 'package:scribe/scribe/ai/domain/repository/ai_scribe_repository.dart';
|
import 'package:scribe/scribe/ai/domain/repository/ai_scribe_repository.dart';
|
||||||
@@ -18,7 +17,6 @@ class GenerateAITextInteractor {
|
|||||||
) async {
|
) async {
|
||||||
try {
|
try {
|
||||||
final prompt = AIPrompts.buildPrompt(action, selectedText);
|
final prompt = AIPrompts.buildPrompt(action, selectedText);
|
||||||
log('$runtimeType::execute():prompt: $prompt');
|
|
||||||
final response = await _repository.generateMessage(prompt);
|
final response = await _repository.generateMessage(prompt);
|
||||||
return Right(GenerateAITextSuccess(response));
|
return Right(GenerateAITextSuccess(response));
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
|
|||||||
@@ -14,9 +14,6 @@ class AICapability extends CapabilityProperties {
|
|||||||
|
|
||||||
Map<String, dynamic> toJson() => _$AICapabilityToJson(this);
|
Map<String, dynamic> toJson() => _$AICapabilityToJson(this);
|
||||||
|
|
||||||
factory AICapability.deserialize(Map<String, dynamic> json) =>
|
|
||||||
_$AICapabilityFromJson(json);
|
|
||||||
|
|
||||||
bool get isScribeEndpointAvailable =>
|
bool get isScribeEndpointAvailable =>
|
||||||
scribeEndpoint?.trim().isNotEmpty == true;
|
scribeEndpoint?.trim().isNotEmpty == true;
|
||||||
|
|
||||||
|
|||||||
@@ -159,16 +159,16 @@ abstract final class AIScribeSizes {
|
|||||||
EdgeInsetsDirectional.all(16);
|
EdgeInsetsDirectional.all(16);
|
||||||
|
|
||||||
static const EdgeInsetsGeometry suggestionInfoPadding =
|
static const EdgeInsetsGeometry suggestionInfoPadding =
|
||||||
EdgeInsetsGeometry.only(bottom: 16);
|
EdgeInsetsDirectional.only(bottom: 16);
|
||||||
|
|
||||||
static const EdgeInsetsGeometry suggestionHeaderPadding =
|
static const EdgeInsetsGeometry suggestionHeaderPadding =
|
||||||
EdgeInsetsGeometry.fromLTRB(16, 8, 8, 8);
|
EdgeInsetsDirectional.only(start: 16, top: 8, end: 8, bottom: 8);
|
||||||
|
|
||||||
static const EdgeInsetsGeometry suggestionFooterPadding =
|
static const EdgeInsetsGeometry suggestionFooterPadding =
|
||||||
EdgeInsetsGeometry.symmetric(horizontal: 16, vertical: 16);
|
EdgeInsetsDirectional.symmetric(horizontal: 16, vertical: 16);
|
||||||
|
|
||||||
static const EdgeInsetsGeometry scribeButtonPadding =
|
static const EdgeInsetsGeometry scribeButtonPadding =
|
||||||
EdgeInsetsGeometry.all(6);
|
EdgeInsetsDirectional.all(6);
|
||||||
|
|
||||||
static const EdgeInsetsGeometry mainActionButtonPadding =
|
static const EdgeInsetsGeometry mainActionButtonPadding =
|
||||||
EdgeInsetsDirectional.symmetric(horizontal: 16, vertical: 8);
|
EdgeInsetsDirectional.symmetric(horizontal: 16, vertical: 8);
|
||||||
|
|||||||
@@ -30,7 +30,7 @@ class AiScribeModalManager {
|
|||||||
submenuController: submenuController,
|
submenuController: submenuController,
|
||||||
),
|
),
|
||||||
barrierColor: AIScribeColors.dialogBarrier,
|
barrierColor: AIScribeColors.dialogBarrier,
|
||||||
).whenComplete(submenuController.hide);
|
).whenComplete(submenuController.dispose);
|
||||||
|
|
||||||
if (aiAction != null) {
|
if (aiAction != null) {
|
||||||
await showAIScribeSuggestionModal(
|
await showAIScribeSuggestionModal(
|
||||||
|
|||||||
@@ -107,10 +107,4 @@ class _AiScribeContextMenuContentState extends State<AiScribeContextMenu> {
|
|||||||
),
|
),
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
@override
|
|
||||||
void dispose() {
|
|
||||||
widget.submenuController?.dispose();
|
|
||||||
super.dispose();
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user